Modify

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#8878 closed defect (fixed)

TestManagerForTracPlugin does not play well with MenusPlugin

Reported by: david.killick@… Owned by: Roberto Longobardi
Priority: normal Component: TestManagerForTracPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

When accessing pages under the Test Manager, we get the error the $ is not a method from the javascript associated with the menus plugin. No other pages give this error.

It seems to be related to the introduction of a second jquery instance at line 571 in wiki.py.

Attachments (1)

wiki.py (28.6 KB) - added by Roberto Longobardi 13 years ago.
Fixed code

Download all attachments as: .zip

Change History (6)

comment:1 Changed 13 years ago by Roberto Longobardi

Hi David, thanks for reporting this and for the debugging hints you give :D

As you may guess I didn't test the plugin with this MenusPlugin, anyway while adding the new jQuery package I feared something wrong would be reported soon :(

I'll take a look during this w/e.

Ciao, Roberto

comment:2 Changed 13 years ago by Roberto Longobardi

Hi David, I found the problem and maybe fixed it.

Please find in attachment the modified wiki.py file, to be replaced inside the "testman4trac/trunk/testmanager" directory.

Build again and replace the plugin, clean your browser's cache and try again.

I've seen this fixes the problem while coexisting with another plugin, so please, let me know if this fixes the problem in your environment.

Ciao, Roberto

Changed 13 years ago by Roberto Longobardi

Attachment: wiki.py added

Fixed code

comment:3 Changed 13 years ago by david.killick@…

Hi Roberto,

This patch seems to have mostly fixed the problem. The menus now render correctly, and there are no JS errors.

On TestManager pages, the menus must be clicked to open then, however, on all other pages the menus open by 'hovering' the mouse over them. Not sure whether this would be a TestManager or a MenusPlugin issue.

comment:4 Changed 13 years ago by Roberto Longobardi

Resolution: fixed
Status: newclosed

Hi David, glad to hear the fix works :D

I've recently made another little change that actually removes the need for a menu in that place, unless you have defined more than one (custom) outcome for any of the color (green, yellow, red).

This should simplify selecting the verdict of a test case manually.

I'll release your fix and the above mentioned change asap, in 1.4.6.

Ciao, Roberto

comment:5 Changed 13 years ago by Roberto Longobardi

Released in 1.4.6.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Roberto Longobardi.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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

 
Note: See TracTickets for help on using tickets.