Modify

Opened 7 years ago

Closed 17 months ago

#2197 closed task (fixed)

Trac 0.11 support

Reported by: tnakani Owned by: rjollos
Priority: high Component: CondFieldsPlugin
Severity: major Keywords:
Cc: Trac Release: 0.11

Description

Hello,
Is this plugin scheduled to support Trac 0.11 ?

Attachments (2)

condfields-0.11.diff (76.1 KB) - added by djs 6 years ago.
Condfields patch for Trac 0.11
condfields-0.11.patch (72.5 KB) - added by djs 6 years ago.
Condfields patch for Trac 0.11, adopting fixes from #2042

Download all attachments as: .zip

Change History (24)

comment:1 Changed 6 years ago by anonymous

  • Priority changed from normal to high
  • Severity changed from normal to major

is there some new information about this issue?
the plugin doesn't work for me with trac 0.11rc1!
are there any dependencies with this new version?

comment:2 follow-up: Changed 6 years ago by coderanger

  • Priority changed from high to normal
  • Severity changed from major to normal

Eventually it may support 0.11, but currently it does not.

comment:3 follow-up: Changed 6 years ago by aurelio

Any updates on 0.11 support since May? Definitely, find the plugin to be valuable and would be willing to help out in the effort if given a few pointers on where to start and the approach to take in support 0.11.

comment:4 in reply to: ↑ 3 ; follow-up: Changed 6 years ago by aurelio

  • Owner changed from coderanger to aurelio
  • Status changed from new to assigned

Replying to aurelio:

Any updates on 0.11 support since May? Definitely, find the plugin to be valuable and would be willing to help out in the effort if given a few pointers on where to start and the approach to take in support 0.11.

Ok, finally got around to get this working under v0.11 of Trac and will submit a patch as soon as I get around to cleaning the changes I made. If anyone wants an alpha already packaged as an egg plugin feel free to message me.

comment:5 follow-up: Changed 6 years ago by dklinger

Coderanger - We would love to try your alpha support of this plugin.

danny.klinger at gmail dot com

comment:6 Changed 6 years ago by lei

Hi Coderanger,

still having the alpha version of 0.11 around?
Could you send me the egg file, I would also try to clean up and publish.

comment:7 Changed 6 years ago by lei

ah, sry, Aurelio *wave*

comment:8 in reply to: ↑ 5 ; follow-up: Changed 6 years ago by dklinger

Replying to dklinger:

Coderanger - We would love to try your alpha support of this plugin.

danny.klinger at gmail dot com

Ooooh - did I ask for this from the wrong person? Sorry Aurelio.

comment:9 in reply to: ↑ 8 ; follow-up: Changed 6 years ago by aurelio

Replying to dklinger:

Replying to dklinger:

Coderanger - We would love to try your alpha support of this plugin.

danny.klinger at gmail dot com

Ooooh - did I ask for this from the wrong person? Sorry Aurelio.

Ah, since I didn't immediately get any messages about the plugin this just eventually fell of my radar, so apologies for the delay. I'll go ahead and e-mail the egg to the provided address. However, I still intend to properly send a patch and get coderanger's approval.

comment:10 in reply to: ↑ 9 Changed 6 years ago by cjb

Hi!

Replying to aurelio:

Ah, since I didn't immediately get any messages about the plugin this just eventually fell of my radar, so apologies for the delay. I'll go ahead and e-mail the egg to the provided address. However, I still intend to properly send a patch and get coderanger's approval.

Would you mind mailing it to me too, please? Happy to help with cleaning it up if you'd like it. :)

comment:11 Changed 6 years ago by cjb

Ah, I should leave an address -- cjb AT laptop DOT org, please. Thanks!

comment:12 in reply to: ↑ 2 Changed 6 years ago by grysle

Replying to coderanger:

Eventually it may support 0.11, but currently it does not.

Coderanger: has this plugin been scheduled to support Trac 0.11? (october 2008)

comment:13 in reply to: ↑ 4 Changed 6 years ago by didoux

Replying to aurelio:

[...]
Ok, finally got around to get this working under v0.11 of Trac and will submit a patch as soon as I get around to cleaning the changes I made. If anyone wants an alpha already packaged as an egg plugin feel free to message me.

I would also be extreemely happy if you can send me this patch... (it would maybee also good to provide it with a link from the project page, no?)... florianseydoux at gmail dot com [thx]

comment:14 Changed 6 years ago by djs

I would also really appreciate access to this patch -- it's the only thing stopping us from migrating to v0.11! djs@…

Changed 6 years ago by djs

Condfields patch for Trac 0.11

comment:15 Changed 6 years ago by anonymous

Attached patch, based on Aurelio's work, is a port to Trac 0.11 and Genshi. Key changes:

  • Use the jquery from Trac, rather than bundling one
  • Provide a static (cacheable) javascript file with most of the functionality
  • Use Genshi transformations to insert the javascript variable assignments into the page header

Changed 6 years ago by djs

Condfields patch for Trac 0.11, adopting fixes from #2042

comment:16 Changed 6 years ago by mixedpuppy

I've added a 0.11 port with an added admin panel in #4668

comment:17 Changed 5 years ago by aurelio

  • Owner changed from aurelio to djs
  • Status changed from assigned to new

Nice changes and patch. djs mind if I reassign the ticket to you? I never did follow up with this ticket as I intended and doubt I'll have more free time to do so in the next few months.

comment:18 Changed 5 years ago by coderanger

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

Already supports 0.11.

comment:19 Changed 4 years ago by anonymous

  • Priority changed from normal to high
  • Resolution fixed deleted
  • Severity changed from normal to major
  • Status changed from closed to reopened

No, it doesn't. This plugin does not work correctly with trac 0.11 for users that do not have TICKET_ADMIN permissions. The consequence is that for standard users some fields show up duplicated when creating new tickets. This must be fixed. A pitty that this plugin did not receive any attention for over a year.

comment:20 Changed 17 months ago by rjollos

  • Owner changed from djs to rjollos
  • Status changed from reopened to new

comment:21 Changed 17 months ago by rjollos

(In [12827]) Refs #2197: Fixed numerous PEP8 style violations. Fixed syntax error in condfields.js.

comment:22 Changed 17 months ago by rjollos

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

It should work with Trac 0.11 and later now, but I've only tested with Trac 1.0.1. Please report any specific issues you have in #9993.

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.