Ticket #10329 (assigned task)

Opened 8 months ago

Last modified 8 months ago

Refactor and redesign

Reported by: rjollos Assigned to: rjollos (accepted)
Priority: normal Component: BacklogPlugin
Severity: normal Keywords:
Cc: anachesa, bart Trac Release: 0.11

Description

Refactoring and redesign will be done against this ticket, for the 0.2 version of the plugin.

Attachments

Change History

09/17/12 07:05:11 changed by rjollos

Just a note to self: backlog.db looks like it was copied from fullblog.db and modified.

09/17/12 08:48:57 changed by rjollos

  • status changed from new to 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".

09/17/12 09:13:17 changed by rjollos

(In [12005]) Refs #10329:

  • Several instances in which the passing of a cursor object was replaced with passing a db object. This avoid potential problems with the cursor being invalid.
  • Removed duplicated "save order"/"reset"/"delete closed" controls on the backlog page.
  • Having BACKLOG_MODIFY* permission now grants BACKLOG_VIEW permission.
  • Removed trailing whitespace and formatted source code.
  • Added encoding and copyright info to top of each file.
  • Added placeholder COPYING file.
  • Added changelog.

09/17/12 09:17:12 changed by rjollos

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.

09/17/12 09:31:38 changed by rjollos

(In [12008]) Refs #10329:

  • 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 backlog was being used in multiple places within the backlog and backlog_list templates, which resulted in all of the elements being movable on the backlog page.
  • 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_OWNER and BACKLOG_ADMIN.

09/17/12 09:39:16 changed by rjollos

(In [12009]) Refs #10329:

  • Added images that should have been included in [12008].
  • Fixed some whitespace and tab/space issues.

Add/Change #10329 (Refactor and redesign)




Change Properties
Action