Ticket #1749: ticketdelete-trac0.11-v2.patch

File ticketdelete-trac0.11-v2.patch, 13.3 kB (added by s0undt3ch, 1 year ago)

Actual patch from svn, no copy/paste

  • ticketdelete/web_ui.py

    old new  
    11# Ticket deleting plugins 
    22 
    33from trac import __version__ as TRAC_VERSION 
    4 from trac import ticket 
    54from trac.core import * 
     5from trac.ticket.model import Ticket 
    66from trac.web.api import IRequestFilter 
    77from trac.web.chrome import ITemplateProvider, add_script, add_stylesheet 
    8 from webadmin.web_ui import IAdminPageProvider 
     8from trac.admin.web_ui import IAdminPanelProvider 
    99import re, traceback, pprint 
    1010from time import strftime, localtime 
    1111 
    …… 
    1414class TicketDeletePlugin(Component): 
    1515    """A small ticket deletion plugin.""" 
    1616     
    17     implements(ITemplateProvider, IAdminPageProvider, IRequestFilter) 
     17    implements(ITemplateProvider, IAdminPanelProvider, IRequestFilter) 
    1818 
    1919    # IRequestFilter methods 
    2020    def pre_process_request(self, req, handler): 
    2121        return handler 
    2222 
    23     def post_process_request(self, req, template, content_type): 
    24         if template == 'ticket.cs' and req.perm.has_permission('TICKET_ADMIN'): 
    25             add_script(req, 'ticketdelete/jquery.js') 
     23    def post_process_request(self, req, template, data, content_type): 
     24        if template == 'ticket.html' and req.perm.has_permission('TICKET_ADMIN'): 
     25            major, minor = self._get_trac_version() 
     26            if major <= 0 and minor < 11: 
     27                add_script(req, 'ticketdelete/jquery.js') 
    2628            add_script(req, 'ticketdelete/ticketdelete.js') 
    2729            add_stylesheet(req, 'ticketdelete/ticketdelete.css') 
    28         return template, content_type 
     30        return template, data, content_type 
    2931  
    30     # IAdminPageProvider methods 
    31     def get_admin_pages(self, req): 
     32    # IAdminPanelProvider methods 
     33    def get_admin_panels(self, req): 
    3234        if req.perm.has_permission('TICKET_ADMIN'): 
    3335            yield ('ticket', 'Ticket System', 'delete', 'Delete') 
    3436            yield ('ticket', 'Ticket System', 'comments', 'Delete Changes') 
    3537             
    36     def process_admin_request(self, req, cat, page, path_info): 
     38    def render_admin_panel(self, req, cat, page, path_info): 
    3739        req.perm.assert_permission('TICKET_ADMIN') 
    3840         
    39         data = {} 
     41        data={} 
     42        ticketdelete = {} 
     43        ticketdelete['href'] = req.href('admin', cat, page) 
     44        ticketdelete['page'] = page 
     45        ticketdelete['redir'] = 1 
    4046 
    41         data['ticketdelete.href'] = req.href('admin', cat, page) 
    42         data['ticketdelete.page'] = page 
    43         data['ticketdelete.redir'] = 1 
    44  
    4547        if req.method == 'POST': 
    4648            if page == 'delete': 
    4749                if 'ticketid' in req.args and 'ticketid2' in req.args: 
    4850                    if req.args.get('ticketid') == req.args.get('ticketid2'): 
    49                         t = self._validate(req, req.args.get('ticketid')) 
     51                        t, ticketdelete = self._validate(req, req.args.get('ticketid'), data) 
    5052                        if t: 
    5153                            self._delete_ticket(t.id) 
    52                             data['ticketdelete.message'] = "Ticket #%s has been deleted." % t.id 
     54                            ticketdelete['message'] = "Ticket #%s has been deleted." % t.id 
    5355                             
    5456                    else: 
    55                         data['ticketdelete.message'] = "The two IDs did not match. Please try again." 
     57                        ticketdelete['message'] = "The two IDs did not match. Please try again." 
    5658            elif page == 'comments': 
    5759                if 'ticketid' in req.args: 
    5860                    req.redirect(req.href.admin(cat, page, req.args.get('ticketid'))) 
    5961                else: 
    60                     t = self._validate(req, path_info) 
     62                    t, ticketdelete = self._validate(req, path_info, ticketdelete) 
    6163                    if t: 
    62                         data['ticketdelete.href'] = req.href('admin', cat, page, path_info) 
     64                        ticketdelete['href'] = req.href('admin', cat, page, path_info) 
    6365                        try: 
    6466                            deletions = None 
    6567                            if "multidelete" in req.args: 
    …… 
    7476                                    ts = int(ts) 
    7577                                    self.log.debug('TicketDelete: Deleting change to ticket %s at %s (%s)'%(t.id,ts,field)) 
    7678                                    self._delete_change(t.id, ts, field) 
    77                                     data['ticketdelete.message'] = "Change to ticket #%s at %s has been modified" % (t.id, strftime('%a, %d %b %Y %H:%M:%S',localtime(ts))) 
    78                                     data['ticketdelete.redir'] = 0 
     79                                    ticketdelete['message'] = "Change to ticket #%s at %s has been modified" % (t.id, strftime('%a, %d %b %Y %H:%M:%S',localtime(ts))) 
     80                                    ticketdelete['redir'] = 0 
    7981                        except ValueError, e: 
    8082                            self.log.debug("TicketDelete: Error is %s"%e) 
    8183                            self.log.debug("TicketDelete: args = '%s'"%req.args.items()) 
    82                             data['ticketdelete.message'] = "Timestamp '%s' not valid" % req.args.get('ts')                     
     84                            ticketdelete['message'] = "Timestamp '%s' not valid" % req.args.get('ts')                     
    8385                     
    8486                 
    8587        if path_info: 
    86             t = self._validate(req, path_info) 
     88            t, ticketdelete = self._validate(req, path_info, ticketdelete) 
    8789            if t: 
    8890                if page == 'comments': 
    8991                    try: 
    …… 
    9698                        data = ticket_data.setdefault(str(time), {}) 
    9799                        data.setdefault('fields', {})[field] = {'old': oldvalue, 'new': newvalue} 
    98100                        data['author'] = author 
    99                         data['prettytime'] = strftime('%a, %d %b %Y %H:%M:%S',localtime(time)) 
     101                        #data['prettytime'] = strftime('%a, %d %b %Y %H:%M:%S',localtime(0.10)) 
    100102                     
    101103                    # Remove all attachment changes                     
    102104                    for k, v in ticket_data.items(): 
    …… 
    107109                    time_list = list(sorted(ticket_data.iterkeys())) 
    108110                    if selected is not None and selected < len(time_list): 
    109111                        ticket_data[time_list[selected]]['checked'] = True 
    110                     data['ticketdelete.changes'] = ticket_data 
     112                    ticketdelete['changes'] = [] 
     113                    for time in time_list: 
     114                        ticketdelete['changes'].append(ticket_data[time]) 
    111115                elif page == 'delete': 
    112                     data['ticketdelete.id'] = t.id 
     116                    ticketdelete['id'] = t.id 
    113117  
    114         return 'ticketdelete_admin.html', data, None 
     118        #data = {'ticketdelete': ticketdelete} 
     119        data['ticketdelete'] = ticketdelete 
    115120 
     121        return 'ticketdelete_admin.html', data 
     122 
    116123    # ITemplateProvider methods 
    117124    def get_templates_dirs(self): 
    118125        """ 
    …… 
    145152        else: 
    146153            return (0,0) 
    147154 
    148     def _validate(self, req, arg): 
     155    def _validate(self, req, arg, ticketdelete): 
    149156        """Validate that arg is a string containing a valid ticket ID.""" 
    150157        try: 
    151158            id = int(arg) 
    152159            t = Ticket(self.env, id) 
    153             return t 
     160            return t, ticketdelete 
    154161        except TracError: 
    155             data['ticketdelete.message'] = "Ticket #%s not found. Please try again." % id 
     162            ticketdelete['message'] = "Ticket #%s not found. Please try again." % id 
    156163        except ValueError: 
    157             data['ticketdelete.message'] = "Ticket ID '%s' is not valid. Please try again." % arg 
    158         return False 
     164            ticketdelete['message'] = "Ticket ID '%s' is not valid. Please try again." % arg 
     165        return False, ticketdelete 
    159166                                                                                                                 
    160167     
    161168    def _delete_ticket(self, id): 

  • ticketdelete/htdocs/ticketdelete.js

    old new  
    77    } else { 
    88        $('#ticket table.properties').after('<div class="description"><h3><span class="inlinebuttons">'+delete_link+'</span>&nbsp;</h3></div>'); 
    99    } 
    10     $('#changelog h3').each(function() { 
     10    $('#changelog form').each(function() { 
     11        if (!/#comment$/.test(this.action)) { 
     12            return; 
     13        } 
    1114        var comment = $('input[@name=replyto]', this)[0]; 
    1215        if (comment) { 
    1316            comment = comment.value; 

  • ticketdelete/templates/ticketdelete_admin.html

    old new  
    1 <div xmlns="http://www.w3.org/1999/xhtml" 
    2   xmlns:py="http://genshi.edgewall.org/" 
    3   id="page"> 
    4   <h2>Delete Ticket<py:if ticketdelete.page="comments"> Changes</py:if></h2> 
     1<!DOCTYPE html 
     2    PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" 
     3    "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd"> 
     4<html xmlns="http://www.w3.org/1999/xhtml" 
     5      xmlns:xi="http://www.w3.org/2001/XInclude" 
     6      xmlns:py="http://genshi.edgewall.org/"> 
     7  <xi:include href="admin.html" /> 
     8  <head> 
     9    <title>Delete</title> 
     10  </head> 
     11<body> 
    512 
     13  <h2>Delete Ticket<py:if test="ticketdelete.page == 'comments'"> Changes</py:if></h2> 
     14 
    615  <py:choose> 
    716    <py:when test="ticketdelete.message and ticketdelete.redir"> 
    817      <b>${ticketdelete.message}</b><br /> 
    …… 
    2231      </form> 
    2332    </py:when> 
    2433    <py:when test="ticketdelete.page == 'comments'"> 
    25       <py:choose has_tickets="len(ticketdelete.changes)"> 
    26         <py:when has_tickets="True"> 
     34      <py:choose test="len(ticketdelete.changes)"> 
     35        <py:when test="True"> 
    2736          <py:if test="ticketdelete.message"> 
    2837            <p><b>${ticketdelete.message}</b></p> 
    2938          </py:if> 
    …… 
    3443            <tbody> 
    3544              <py:for each="change in ticketdelete.changes"> 
    3645                <tr> 
    37                   <td><input type="checkbox" name="dontcare" value="dontcare" id="checkbox_${change}" checked="${change.checked or None}" /></td> 
     46                  <td><input type="checkbox" name="dontcare" value="dontcare" id="checkbox_${change}" checked="${change.checked or None}" onchange="syncchecks(this)" /></td> 
    3847                  <td colspan="3"><b>Change at ${change.prettytime} by ${change.author}</b></td> 
    3948                  <td><input type="submit" name="delete_${change}" value="Delete change" /></td> 
    4049                  <tr> 
    4150                    <py:for each="field in change.fields"> 
    4251                      <tr> 
    43                         <td><input type="checkbox" id="checkbox${field}_${change}" name="mdelete" value="${field}_${change}" checked="${change.checked or None}" /></td> 
     52                        <td><input type="checkbox" id="checkbox_${change}_${field}" name="mdelete" value="${field}_${change}" checked="${change.checked or None}" onchange="if(this.checked) { $('#checkbox_${change}').checked = 0; }" /></td> 
     53 
    4454                        <td>${field}</td> 
    45                         <py:choose test="name(field)"> 
     55                        <py:choose test="str(field)"> 
    4656                          <td py:when="comment" colspan="2">${field.new}</td> 
    4757 
    4858                          <py:otherwise> 
    …… 
    6171         
    6272        <script type="text/javascript"> 
    6373        <!-- 
    64             function toggleboxen(me, boxen)  
     74            function syncchecks(me)  
    6575            { 
    66                 status = document.getElementById("checkbox_" + me).checked; 
    67                 boxen.pop() // Remove the last (blank) entry. 
    68                 for (box in boxen) { 
    69                     //alert("Changing checkbox"+boxen[box]+"_"+me); 
    70                     document.getElementById("checkbox"+boxen[box]+"_"+me).checked = status; 
     76                var checked = me.checked; 
     77                var checks = document.getElementsByTagName("input"); 
     78                var re = new RegExp("^" + me.id + "_"); 
     79                for (var i = 0, l = checks.length; i < l; i ++) { 
     80                    var check = checks[i]; 
     81                    if (check.type == "checkbox" && re.test(check.id)) { 
     82                      check.checked = checked; 
     83                    } 
    7184                } 
    7285            } 
    73              
    74             <py:for each="change in ticketdelete.changes"> 
    75             addEvent(document.getElementById("checkbox_${change}"), "change", function() { 
    76                 var boxen = Array(${py:for each="field in change.fields"${field}",</py:for>""); 
    77                 toggleboxen("${change}", boxen); //Array(<py:for each="field in change.fields">"${field}",</py:for>)); 
    78             }); 
    79             <py:for each="field in change.fields"> 
    80               addEvent(document.getElementById("checkbox${field}_${change}"),"change", function() { 
    81                   if(!document.getElementById("checkbox${field}_${change}").checked) { 
    82                   document.getElementById("checkbox${change}").checked = 0; 
    83                 } 
    84             }); 
    85             </py:for> 
    86             </py:for> 
    87  
    88  
    8986        //--> 
    9087        </script> 
    9188 
    …… 
    104101      </py:choose> 
    105102    </py:when> 
    106103  </py:choose> 
    107 </div> 
     104</body> 
     105 
     106</html>