Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Banner edit links did not work correctly #8

Open
cat-in-136 opened this issue Mar 21, 2023 · 0 comments · May be fixed by #9
Open

Banner edit links did not work correctly #8

cat-in-136 opened this issue Mar 21, 2023 · 0 comments · May be fixed by #9

Comments

@cat-in-136
Copy link

cat-in-136 commented Mar 21, 2023

Summary

Banner edit links did not work correctly

Description

Clicking the edit link on the banner navigates the edit page /global_banner/1234 opens (where 1234 seems to be :id of the origin page). When updating the banner from here, the POST to /global_banner/1234 is invoked, but the URL does not accept POST, resulting cause a 404 error.

Banner edit page from the Redmine admin page navigates /global_banner/ (without :id number). This issue is not reproduced from the edit page without :id number.

Environment

  • Redmine version: 5.0.5
  • Installed plugins: confirmed that this issue is reproduced with only this plugin installed.
  • Ruby version: 3.1.3-p185
  • OS Platform: linux
  • Database (MariaDB, MySQL, PostgreSQL) and its version: PostgreSQL, SQLite
  • Rails Env

Visual Proof / Screenshot

Go to some issue page. Here is http://example.com/issues/15
001

Click "Edit" link to navigate to http://example.com/global_banner/15
002

Click "Apply" button causes 404 Error.

Expected Results

Update banner

Actual Results

404 Not Found

Note that it is caused by routing error. If debug mode, you can see routing error page:
003

Workaround

There are some workarounds:

  • update the banner from Redmine admin page
  • modify edit link URL on the banner using view_customize
  • modify edit link URL on the banner with source code modification
cat-in-136 added a commit to cat-in-136/redmine_banner that referenced this issue Mar 21, 2023
…anner edit links did not work correctly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant