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

[configuration] No error when renaming a subproject to a name that already exists #7947

Open
ridz1208 opened this issue Jan 11, 2022 · 1 comment
Assignees
Labels
24.0.0-bugs Issues or bug fix PRs that were raised during the testing of release 24.0.0 Category: Bug PR or issue that aims to report or fix a bug Module: configuration PR or issue related to configuration module State: Discussion required PR or issue that requires the resolution of a discussion with the relevant parties to proceed
Milestone

Comments

@ridz1208
Copy link
Collaborator

Describe the bug
If you rename an existing subproject to a name that is already in use by another subproject, a 500 Error is thrown in the apache error log but no feedback is given to the user, instead the page is simply not updated, not freshed and the change seems to have taken effect until you reload the module

image

To Reproduce
Steps to reproduce the behavior (attach screenshots if applicable):

  1. Go to the Configuration module -> Subprojects
  2. chose any subproject and edit its name to match another name in use
  3. hit submit

What did you expect to happen?
An error should be thrown saying that the subproject name is already in use

@ridz1208 ridz1208 added Category: Bug PR or issue that aims to report or fix a bug 24.0.0-bugs Issues or bug fix PRs that were raised during the testing of release 24.0.0 labels Jan 11, 2022
@ridz1208 ridz1208 added the State: Discussion required PR or issue that requires the resolution of a discussion with the relevant parties to proceed label Jan 17, 2023
@ridz1208
Copy link
Collaborator Author

@maltheism you can do this in your sleep... but please don't.

  • handle the error in the back end
  • swal the error in the front end

@CamilleBeau CamilleBeau assigned KLaFleur and unassigned samirdas and maltheism Sep 24, 2024
@driusan driusan added the Module: configuration PR or issue related to configuration module label Jan 21, 2025
@driusan driusan added this to the 27.0.0 milestone Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
24.0.0-bugs Issues or bug fix PRs that were raised during the testing of release 24.0.0 Category: Bug PR or issue that aims to report or fix a bug Module: configuration PR or issue related to configuration module State: Discussion required PR or issue that requires the resolution of a discussion with the relevant parties to proceed
Projects
Status: No status
Development

No branches or pull requests

5 participants