Merge pull request #17347 from opf/bug/59865-trailing-in-journal-diff…

…-parameter-activity_page-breaks-page

Improve robustness of activity back url

app/helpers/journals_helper.rb

@@ -38,7 +38,8 @@ def back_to_activity_page_url(activity_page)
     in ["users", user_id]
       user_url(user_id)
     in ["work_packages", work_package_id]
-      work_package_url(work_package_id)
+      # Sometimes the parameter provided is erroneous (having an extra ') for unknown reasons.
+      work_package_url(work_package_id.chomp("'"))
     else
       nil
     end

spec/helpers/journals_helper_spec.rb

@@ -38,6 +38,8 @@
       "unsupported_gizmo" => nil,
       "users/5" => "http://test.host/users/5",
       "work_packages/42" => "http://test.host/work_packages/42",
+      # Sometimes the url is erroneous (having an extra ') for unknown reasons.
+      "work_packages/42'" => "http://test.host/work_packages/42",
       nil => nil
     }.each do |activity_page, expected_url|
       context "when activity page is #{activity_page.inspect}" do