Merge pull request #460 from topcoder-platform/issues-458

jmgasper · web-flow · commit 4a8afa4991e3 · 2021-03-06T12:14:15.000+11:00
Issues-458: fixed target url in Discussion Options Dropdown
diff --git a/vanilla/applications/vanilla/views/discussion/helper_functions.php b/vanilla/applications/vanilla/views/discussion/helper_functions.php
@@ -336,7 +336,7 @@ function getDiscussionOptionsDropdown($discussion = null) {
         }
 
         $discussionID = $discussion->DiscussionID;
-        $categoryUrl = urlencode(categoryUrl(CategoryModel::categories($categoryID)));
+        $categoryUrl = urlencode(categoryUrl(CategoryModel::categories($categoryID),'',false));
 
         // Permissions
         $canEdit = DiscussionModel::canEdit($discussion, $timeLeft);

Original file line number	Diff line number	Diff line change
`@@ -336,7 +336,7 @@ function getDiscussionOptionsDropdown($discussion = null) {`
`336`	`336`	`}`
`337`	`337`
`338`	`338`	`$discussionID = $discussion->DiscussionID;`
`339`		`- $categoryUrl = urlencode(categoryUrl(CategoryModel::categories($categoryID)));`
	`339`	`+ $categoryUrl = urlencode(categoryUrl(CategoryModel::categories($categoryID),'',false));`
`340`	`340`
`341`	`341`	`// Permissions`
`342`	`342`	`$canEdit = DiscussionModel::canEdit($discussion, $timeLeft);`