*****************************************************************
***** *****
***** !!!! THIS BUG TRACKER IS FOR GERRIT CODE REVIEW !!!! *****
***** *****
***** DO NOT SUBMIT BUGS FOR CHROME, ANDROID, CYANOGENMOD, *****
***** INTERNAL ISSUES WITH YOUR COMPANY'S GERRIT SETUP, ETC.*****
***** *****
***** THOSE ISSUES BELONG IN DIFFERENT ISSUE TRACKERS *****
***** *****
*****************************************************************
Affected Version:
What steps will reproduce the problem?
1. Install a plugin that validates the format of commits
1.1 That validation must consider "Edit Project Config" an invalid commit message
2. Log in
3. Click on Projects
5. Click on General
6. Click on "Edit Config"
What is the expected output? What do you see instead?
The configuration page should be opened so that the user can edit as desired
Please provide any additional information below.
Instead, an "500 - Internal" error happens because "Edit Config" button generates a commit with a fixed commit message ("Edit Project Config")
As commit messages may follow a strict policy, by clicking "Edit Config" message should allow the user to chose which message to use in the commit. It might suggest "Edit Project Config", but the user may decide to change it. Another approach would be commit only after changes are made. At that point, a commit message would be requested. That is good to document why that config change was done in the first place
For now I work this issue around by forcing my plugin to accept "Edit Project Config" as a valid commit message - but that is ugly :)