Skip to content

Commit

Permalink
fix: #8699 tags route is case sensitive, though tags are not
Browse files Browse the repository at this point in the history
  • Loading branch information
julianlam committed Sep 30, 2020
1 parent 2a1da61 commit 38f88fc
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/controllers/tags.js
Expand Up @@ -3,17 +3,19 @@
const validator = require('validator');
const nconf = require('nconf');

const meta = require('../meta');
const user = require('../user');
const categories = require('../categories');
const topics = require('../topics');
const privileges = require('../privileges');
const pagination = require('../pagination');
const utils = require('../utils');
const helpers = require('./helpers');

const tagsController = module.exports;

tagsController.getTag = async function (req, res) {
const tag = validator.escape(String(req.params.tag));
const tag = validator.escape(utils.cleanUpTag(req.params.tag, meta.config.maximumTagLength));
const page = parseInt(req.query.page, 10) || 1;

const templateData = {
Expand All @@ -27,8 +29,8 @@ tagsController.getTag = async function (req, res) {
const stop = start + settings.topicsPerPage - 1;
const states = [categories.watchStates.watching, categories.watchStates.notwatching, categories.watchStates.ignoring];
const [topicCount, tids, categoriesData] = await Promise.all([
topics.getTagTopicCount(req.params.tag),
topics.getTagTids(req.params.tag, start, stop),
topics.getTagTopicCount(tag),
topics.getTagTids(tag, start, stop),
helpers.getCategoriesByStates(req.uid, '', states),
]);

Expand Down

0 comments on commit 38f88fc

Please sign in to comment.