Bug: edit-in-place project title corruption #21
Comments
|
This is using Firefox 23.0 on Linux Mint 13, KDE 4.8.5. |
|
I can confirm the fault with other browsers and in other lists (which is unsurprising, in that it'd be particularly odd were only the Projects list to be affected). I can't remember the details of how Safe In Place Editor posts back its updates to values, but this may have arisen from one of a few recent Rails patches related to parameter parsing vulnerabilities. Since I can easily replicate this on my development machine it should be straightforward diagnose and fix. |
pond
added a commit
that referenced
this issue
Oct 15, 2013
Fixed issues #21, #22 and #24. Added many tests to work towards satisfying issue #6 and allow third parties to better verify their database operation. A lot of test data is included which bulks the distribution substantially. Properly implemented access control and in place editing for saved reports; also included reports in audit table data. Reports include optimistic locks now, so that if two users were to edit the same thing simultaneously, the second user to submit the changes would be notified. This is unlikely but a worthwhile and easy addition. Database changes: OpenID URLs are now stored canonicalised in the database and the front-end is much more relaxed in what it accepts. Work packets had a description field but it was never used so has been removed, reducing the database storage requirements. Their commit dates are now simply Dates, not Date/Time objects, which solves issues with database compatibility and simplifies code slightly in a few places. SQLite blank template for "database.yml" included in addition to PostgreSQL. "README.rdoc" mentions this database and provides configuration advice. Misc changes: Fixed timesheet bulk commit bug that stopped it presenting the commit form when certain date conditions arose. Improved audit list model name display. Small efficiency improvement in Control Panel. Modern scope syntax used in Project, with additional validation revealed as necessary by new tests. New useful scopes added to Task for billable and not-billable hours and updated to modern syntax; much more efficient work packet processing therein. Timesheet access permission model tweaked in response to test results; more logical/consistent now (see "README.rdoc"). Test-driven changes to TimesheetRow for robustness in hypothetical future code use cases. Modern scope syntax in User model and some new scopes to aid tests; "has_many" relationship to saved reports, previously overlooked, now added. Dead code stripped from WorkPacket and TrackRecordSections. Documentation updated, especially the main "README.rdoc" file, which should make life easier for new users.
|
Fixed by 0db4015. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When editing a project name in place in the list of projects, all spaces get replaced with + signs. This does not happen if you go to the 'edit project' page.
For example: edit "Foo Monitor: Phase 1" to "Foo Monitor: Phase 2" results in the project being called "Foo+Monitor:+Phase+2".
The text was updated successfully, but these errors were encountered: