Ticket #1749 (closed enhancement: fixed)

Opened 6 years ago

Last modified 10 months ago

[Patch] 0.11 support for TicketDeletePlugin

Reported by: phpxcache@gmail.com Assigned to: asmodai
Priority: normal Component: TicketDeletePlugin
Severity: blocker Keywords:
Cc: benoitg@coeus.ca, datallah@pidgin.im, macke@yar.nu, listman@burble.net Trac Release: 0.11

Description

0.11 support for deleteticket

Attachments

ticketdelete-trac0.11.patch (13.3 kB) - added by phpxcache on 07/07/07 12:34:31.
ticketdelete-trac0.11-v2.patch (13.3 kB) - added by s0undt3ch on 07/10/07 19:21:05.
Actual patch from svn, no copy/paste
ticketdelete-trac0.11-v3.patch (12.6 kB) - added by hkaimio on 12/06/07 13:20:12.
new version, tested with trac 0.11 r6202
ticketdelete-trac0.11-v4.patch (7.4 kB) - added by kkckkc on 04/17/08 13:19:48.
Patch tested against r3519
ticketdeleteplugin_timestamp_fix.patch (1.2 kB) - added by jonas@edgewall.com on 04/22/08 23:35:47.
Asmodai this appears to fix the timestamp issue on t.e.o we talked about yesterday
trac-0.11-okamototk.patch (8.0 kB) - added by anonymous on 07/28/08 02:14:25.
patch for 0.11 by okamototk
ticketdeleteplugin_0.11_working.patch (62.1 kB) - added by daniel on 08/03/08 09:01:09.
Patch to implement the button with a ITemplateStreamFilter and make the plugin work fully

Change History

07/07/07 12:34:31 changed by phpxcache

  • attachment ticketdelete-trac0.11.patch added.

07/07/07 12:35:49 changed by phpxcache

  • release changed from 0.10 to 0.11.

07/10/07 19:21:05 changed by s0undt3ch

  • attachment ticketdelete-trac0.11-v2.patch added.

Actual patch from svn, no copy/paste

(follow-up: ↓ 3 ) 07/10/07 19:23:03 changed by s0undt3ch

I re-added the v2 patch because the first one was a copy/paste on a remote machine and might introduce apply problems.

(in reply to: ↑ 2 ) 07/12/07 15:53:24 changed by s0undt3ch

  • type changed from defect to enhancement.
  • severity changed from normal to blocker.

Replying to s0undt3ch:

I re-added the v2 patch because the first one was a copy/paste on a remote machine and might introduce apply problems.

This version still fails to delete ticket changes, ie, it only works for deleting complete tickets.

12/06/07 13:20:12 changed by hkaimio

  • attachment ticketdelete-trac0.11-v3.patch added.

new version, tested with trac 0.11 r6202

12/06/07 13:21:10 changed by hkaimio

New ersion of the patch that works with the current 0.11 tree.

01/24/08 18:58:52 changed by anonymous

i install deleteticket on trac 0.11 via

easy_install http://trac-hacks.org/svn/ticketdeleteplugin/0.11/

Installation Works but no menu on the Admin page and i have the following error on the log:

ERROR: Skipping "ticketdelete.web_ui = ticketdelete.web_ui": (can't import "No module named webadmin.web_ui")

Thanks for Helping

02/04/08 18:38:34 changed by ross

hkaimio,

even with your latest patch ticketdelete-trac0.11-v3.patch it still does not work on 0.11 for deleting ticket changes:

Trac detected an internal error: 
TypeError: object of type 'Undefined' has no len()


File "C:\Documents and Settings\LocalService\Application Data\Python-Eggs\tracticketdelete-2.0-py2.5.egg-tmp\ticketdelete\templates\ticketdelete_admin.html", line 34, in <Expression u'len(ticketdelete.changes)'> 

Would be great if someone puts the fully working TicketDeletePlugin for 0.11 It's really very useful.

02/25/08 16:45:06 changed by anonymous

  • cc set to benoitg@coeus.ca.

03/22/08 03:41:21 changed by gregoryba@gmail.com

  • cc changed from benoitg@coeus.ca to benoitg@coeus.ca, gregoryba@gmail.com.

there's a traceback of an error with 0.11 over at #2781, might or might not be related.

04/17/08 13:19:48 changed by kkckkc

  • attachment ticketdelete-trac0.11-v4.patch added.

Patch tested against r3519

04/17/08 13:21:32 changed by kkckkc

I've attached a patch against r3519 that seem to be working fine with 0.11

The path also enables some attachment deletion, but use this with care. I haven't really tested this

04/17/08 21:15:25 changed by asmodai

  • status changed from new to assigned.
  • owner changed from coderanger to asmodai.

Committed patch v4 in r3526.

04/22/08 23:35:47 changed by jonas@edgewall.com

  • attachment ticketdeleteplugin_timestamp_fix.patch added.

Asmodai this appears to fix the timestamp issue on t.e.o we talked about yesterday

04/23/08 21:30:43 changed by asmodai

Committed in r3542.

04/23/08 21:31:43 changed by asmodai

Err, make that r3543.

07/28/08 02:13:34 changed by anonymous

TicketDeletePlugin doesn't work yes.

I attached additional patch for 0.11(trac-0.11-okamototk.patch).

regards,

Takashi Okamoto

07/28/08 02:14:25 changed by anonymous

  • attachment trac-0.11-okamototk.patch added.

patch for 0.11 by okamototk

08/03/08 09:01:09 changed by daniel

  • attachment ticketdeleteplugin_0.11_working.patch added.

Patch to implement the button with a ITemplateStreamFilter and make the plugin work fully

(follow-up: ↓ 16 ) 08/03/08 09:11:33 changed by datallah

I attached a patch which brings this plugin fully up to date with 0.11 as far as I can tell. Both the overall ticket deletion and the change deletion pages appear to work correctly, including linking from Delete buttons on the ticket screen.

I wish I had seen this ticket before starting working on this as it would have saved me some time.

Some of the changes I made in addition to the bug fixes:

  • The adding of the buttons has been updated to a ITemplateStreamFilter to !add them as the page is being rendered on the server instead of adding them with javascript. (thanks to aat for helping me figure out how to do this)
  • Make the "back" button link to the referrer screen instead of always the !admin screen

08/03/08 09:54:30 changed by datallah

  • cc changed from benoitg@coeus.ca, gregoryba@gmail.com to benoitg@coeus.ca, gregoryba@gmail.com, datallah@pidgin.im.

(in reply to: ↑ 14 ) 02/09/09 15:58:49 changed by otaku42

Replying to datallah:

I attached a patch which brings this plugin fully up to date with 0.11 as far as I can tell.

I can confirm that this patch works in combination with r3543.

(follow-up: ↓ 18 ) 02/09/09 21:23:52 changed by anonymous

Doesn't work with 0.11-stable (and Mastertickets?)

(in reply to: ↑ 17 ) 02/09/09 21:28:35 changed by datallah@pidgin.im

Replying to anonymous:

Doesn't work with 0.11-stable (and Mastertickets?)

What about it doesn't work?

02/09/09 23:10:51 changed by gregoryba@gmail.com

  • cc changed from benoitg@coeus.ca, gregoryba@gmail.com, datallah@pidgin.im to benoitg@coeus.ca, datallah@pidgin.im.

removed my cc.

07/29/09 03:22:26 changed by rjollos

Hello, I am having a few problems with the plug-in in 0.11. These are described in ticket #996 and #5580. I've been looking at this thread and trying to figure out if all of the patches here have been committed to the 0.11 branch, but I'm a bit lost. Can someone tell me if the 0.11 branch is the 'best' working version with 0.11?

(follow-up: ↓ 22 ) 08/18/09 13:01:33 changed by marcusl

  • cc changed from benoitg@coeus.ca, datallah@pidgin.im to benoitg@coeus.ca, datallah@pidgin.im, macke@yar.nu.

Add me to cc, has similar 0.11 problems.

(in reply to: ↑ 21 ) 08/31/09 04:51:20 changed by anonymous

  • cc changed from benoitg@coeus.ca, datallah@pidgin.im, macke@yar.nu to benoitg@coeus.ca, datallah@pidgin.im, macke@yar.nu, listman@burble.net.

Replying to marcusl:

Add me to cc, has similar 0.11 problems.

add me to please. this plugin seems to be broken in 0.11

10/22/09 19:15:31 changed by k0s

can you see if this works with the 0.11 branch since r6812? its working on 0.11.5 here fine

10/22/09 22:13:54 changed by anonymous

  • summary changed from 0.11 support for deleteticket to [Patch] 0.11 support for TicketDeletePlugin.

10/29/09 19:28:35 changed by rjollos

See also #3649.

10/29/09 20:01:11 changed by rjollos

See also #1114.

03/19/11 10:00:03 changed by johannes

Hi everybody,

I am having this problem: installed the ticket delete egg under 0.11b and the log is giving this:

2011-03-19 09:27:52,750 Trac[loader] ERROR: Skipping "ticketdelete.web_ui = ticketdelete.web_ui": (can't import "cannot import name add_notice")

Is this related to the fix you did here? If so how can I merge the svn-sources and the patch? I am working under WinXP and I downloaded the patch.exe from Gnu, but this is giving me the following as result:

#####output of win console######
C:\bla>c:\Programme\GnuWin32\bin\patch.exe -p0 -i ticketdeleteplugin_0.11_working.patch
patching file ticketdelete/web_ui.py
Assertion failed: hunk, file ../patch-2.5.9-src/patch.c, line 354

This application has requested the Runtime to terminate it in an unusual way. Please contact the application's support team for more information.

###### end console ###### can Ido it with svn merge, too?

Thanks for your help!

Best Regards, Johannes

03/19/11 10:49:52 changed by johannes

Hi all,

I just upgraded to 0.12

best J

07/26/12 23:27:26 changed by matthijs

I've been using ticketdelete with 0.11 for quite some time (years) now, so I guess this ticket is obsolete?

07/27/12 02:57:17 changed by rjollos

#2781 closed as a duplicate.

07/27/12 02:59:52 changed by rjollos

#6289 is closed as a duplicate, since switching to ITemplateStreamFilter implementation will certainly fix the issue.

07/27/12 03:03:55 changed by rjollos

#2870 closed as a duplicate.

07/27/12 04:54:24 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

(In [11802]) Refs #1749: (3.0.0dev) Applied patch by Daniel with modifications. Delete buttons are added in ITemplateStreamFilter.filter_stream rather than using JavaScript?.

07/27/12 04:54:50 changed by rjollos

Please create a new ticket for any issues found while testing r11802. Feedback is appreciated.

07/27/12 05:09:18 changed by rjollos

[11802] should resolve #7113, #8572, #6442, #6289 and issues with any plugins that use JQuery or JavaScript.


Add/Change #1749 ([Patch] 0.11 support for TicketDeletePlugin)




Change Properties
Action