Opened 3 years ago

Closed 2 months ago

Last modified 2 months ago

#11632 closed enhancement (fixed)

Simplify code by using add_script_data

Reported by: rjollos Owned by: rjollos
Priority: normal Component: QueuesPlugin
Severity: normal Keywords:
Cc: justinludwig Trac Release:


We can simplify the code quite a bit by removing the Genshi markup from the JavaScript template, moving the file from templates to htdocs and using add_script_data to pass the variables.

Also, I think the plugin doesn't currently work with Trac 1.0. It seemed to be working fine in 0.12, but when I switched branches the re-ordering would no longer be saved. I didn't see any errors in the console, but didn't dig any deeper into the issue.

Attachments (0)

Change History (5)

comment:1 Changed 2 months ago by rjollos

  • Cc justinludwig added

comment:2 Changed 2 months ago by rjollos

  • Owner changed from robguttman to rjollos
  • Status changed from new to accepted

comment:3 Changed 2 months ago by rjollos

  • Resolution set to fixed
  • Status changed from accepted to closed

In 15979:

1.2.0dev: Replace virtual script by using add_script_data

Fixes #11632.

comment:4 follow-up: Changed 2 months ago by rjollos

@justinludwig: I implemented this pretty quickly. It seems to be working well, but please report back if you are able to do more testing. You'll need to install from queuesplugin/1.2.

comment:5 in reply to: ↑ 4 Changed 2 months ago by justinludwig

I tried out the latest from the queuesplugin/1.2 branch -- works great, thanks!

Add Comment

Modify Ticket

as closed The owner will remain rjollos.
The resolution will be deleted. Next status will be 'reopened'.

E-mail address and user name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.