FIX: Group name with capital letter throws 404 error (#73)

discourse · Jul 10, 2020 · 43759bd · 43759bd · discoursereviewbot · Jul 11, 2020
1 parent 48cda7b
commit 43759bd
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/assets/javascripts/discourse/templates/group/assignments.hbs b/assets/javascripts/discourse/templates/group/assignments.hbs
@@ -1,4 +1,3 @@
-{{log model}}
 <section class="user-secondary-navigation">
     {{#mobile-nav class="activity-nav" desktopClass="action-list activity-list nav-stacked" currentPath=router._router.currentPath}}
       {{#load-more selector=".activity-nav li" action=(action "loadMore")}}

diff --git a/plugin.rb b/plugin.rb
@@ -250,7 +250,7 @@ class ::ListController
   add_to_class(:list_controller, :group_topics_assigned) do
     page = (params[:page].to_i || 0).to_i
 
-    group = Group.find_by("LOWER(name) = ?", params[:groupname])
+    group = Group.find_by("name = ?", params[:groupname])
     raise Discourse::NotFound unless group
     raise Discourse::InvalidAccess unless current_user.can_assign?