Modify ↓
Opened 13 years ago
Closed 8 years ago
#10329 closed task (fixed)
Refactor and redesign
| Reported by: | Ryan J Ollos | Owned by: | Ryan J Ollos |
|---|---|---|---|
| Priority: | normal | Component: | BacklogPlugin |
| Severity: | normal | Keywords: | |
| Cc: | Anna Nachesa, Bart Ogryczak | Trac Release: | 0.11 |
Description
Refactoring and redesign will be done against this ticket, for the 0.2 version of the plugin.
Attachments (0)
Change History (8)
comment:1 Changed 13 years ago by
comment:2 Changed 13 years ago by
| Status: | new → assigned |
|---|
The following error is seen in the javascript error console:
Resource interpreted as Script but transferred with MIME type text/x-javascript: "http://localhost:8000/tracdev/chrome/backlog/js /jquery-ui-1.7.2.custom.min.js".
comment:3 Changed 13 years ago by
- Several instances in which the passing of a
cursorobject was replaced with passing adbobject. This avoid potential problems with thecursorbeing invalid. - Removed duplicated "save order"/"reset"/"delete closed" controls on the backlog page.
- Having
BACKLOG_MODIFY*permission now grantsBACKLOG_VIEWpermission. - Removed trailing whitespace and formatted source code.
- Added encoding and copyright info to top of each file.
- Added placeholder
COPYINGfile. - Added changelog.
comment:4 Changed 13 years ago by
I see this error even when the user has the specified permission:
BACKLOG_MODIFY_PRODUCT_AND_COMMUNITY privileges are required to perform this operation. You don't have the required permissions.
Inspecting the perm._cache field, the user has only the BACKLOG_VIEW permission. I think there is something fundamentally wrong with how the permissions are generated. Besides, generating numerous permissions like this is not the way to implement fine-grained permissions. I'll eliminate the per-backlog permission for now, but they will be restored later on.
comment:5 Changed 13 years ago by
- Upgraded jQuery UI from 1.7.2 to 1.7.3, which resolved the mimetype issue described in comment:2. It is unlikely the upgrade itself fixed the issue; rather it may have been resolved simply by including a clean instance of jQuery UI 1.7.2.
- FIX: The form controls were moved outside of the enclosing form elements in [12005], which caused them to stop working.
- FIX: The class
backlogwas being used in multiple places within thebacklogandbacklog_listtemplates, which resulted in all of the elements being movable on thebacklogpage. - The dynamically-generated permission set was causing problems described in comment:4. In the future, fine-grained permissions will be implemented as part of the redesign. For now, the permission set is
BACKLOG_VIEW,BACKLOG_MODIFY,BACKLOG_OWNERandBACKLOG_ADMIN.
comment:6 Changed 13 years ago by
comment:7 Changed 12 years ago by
| Status: | assigned → new |
|---|
comment:8 Changed 8 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.



Just a note to self:
backlog.dblooks like it was copied fromfullblog.dband modified.