| 1 | from trac.ticket import ITicketChangeListener, Ticket, ITicketManipulator |
|---|
| 2 | from trac.core import * |
|---|
| 3 | import datetime |
|---|
| 4 | |
|---|
| 5 | def identity(x): |
|---|
| 6 | return x; |
|---|
| 7 | |
|---|
| 8 | def convertfloat(x): |
|---|
| 9 | "some european countries use , as the decimal separator" |
|---|
| 10 | x = str(x).strip() |
|---|
| 11 | if len(x) > 0: |
|---|
| 12 | return float(x.replace(',','.')) |
|---|
| 13 | else: |
|---|
| 14 | return 0.0 |
|---|
| 15 | |
|---|
| 16 | |
|---|
| 17 | try: |
|---|
| 18 | import trac.util.datefmt |
|---|
| 19 | to_timestamp = trac.util.datefmt.to_timestamp |
|---|
| 20 | except Exception: |
|---|
| 21 | to_timestamp = identity |
|---|
| 22 | |
|---|
| 23 | |
|---|
| 24 | def save_custom_field_value( db, ticket_id, field, value ): |
|---|
| 25 | cursor = db.cursor(); |
|---|
| 26 | cursor.execute("SELECT * FROM ticket_custom " |
|---|
| 27 | "WHERE ticket=%s and name=%s", (ticket_id, field)) |
|---|
| 28 | if cursor.fetchone(): |
|---|
| 29 | cursor.execute("UPDATE ticket_custom SET value=%s " |
|---|
| 30 | "WHERE ticket=%s AND name=%s", |
|---|
| 31 | (value, ticket_id, field)) |
|---|
| 32 | else: |
|---|
| 33 | cursor.execute("INSERT INTO ticket_custom (ticket,name, " |
|---|
| 34 | "value) VALUES(%s,%s,%s)", |
|---|
| 35 | (ticket_id, field, value)) |
|---|
| 36 | db.commit() |
|---|
| 37 | |
|---|
| 38 | DONTUPDATE = "DONTUPDATE" |
|---|
| 39 | |
|---|
| 40 | def save_ticket_change( db, ticket_id, author, change_time, field, oldvalue, newvalue, log, dontinsert=False): |
|---|
| 41 | """tries to save a ticket change, |
|---|
| 42 | |
|---|
| 43 | dontinsert means do not add the change if it didnt already exist |
|---|
| 44 | """ |
|---|
| 45 | if type(change_time) == datetime.datetime: |
|---|
| 46 | change_time = to_timestamp(change_time) |
|---|
| 47 | cursor = db.cursor(); |
|---|
| 48 | sql = """SELECT * FROM ticket_change |
|---|
| 49 | WHERE ticket=%s and author=%s and time=%s and field=%s""" |
|---|
| 50 | |
|---|
| 51 | cursor.execute(sql, (ticket_id, author, change_time, field)) |
|---|
| 52 | if cursor.fetchone(): |
|---|
| 53 | if oldvalue == DONTUPDATE: |
|---|
| 54 | cursor.execute("""UPDATE ticket_change SET newvalue=%s |
|---|
| 55 | WHERE ticket=%s and author=%s and time=%s and field=%s""", |
|---|
| 56 | ( newvalue, ticket_id, author, change_time, field)) |
|---|
| 57 | |
|---|
| 58 | else: |
|---|
| 59 | cursor.execute("""UPDATE ticket_change SET oldvalue=%s, newvalue=%s |
|---|
| 60 | WHERE ticket=%s and author=%s and time=%s and field=%s""", |
|---|
| 61 | (oldvalue, newvalue, ticket_id, author, change_time, field)) |
|---|
| 62 | else: |
|---|
| 63 | if oldvalue == DONTUPDATE: |
|---|
| 64 | oldvalue = '0' |
|---|
| 65 | if not dontinsert: |
|---|
| 66 | cursor.execute("""INSERT INTO ticket_change (ticket,time,author,field, oldvalue, newvalue) |
|---|
| 67 | VALUES(%s, %s, %s, %s, %s, %s)""", |
|---|
| 68 | (ticket_id, change_time, author, field, oldvalue, newvalue)) |
|---|
| 69 | db.commit() |
|---|
| 70 | |
|---|
| 71 | class TimeTrackingTicketObserver(Component): |
|---|
| 72 | implements(ITicketChangeListener) |
|---|
| 73 | def __init__(self): |
|---|
| 74 | pass |
|---|
| 75 | |
|---|
| 76 | def watch_hours(self, ticket): |
|---|
| 77 | |
|---|
| 78 | def readTicketValue(name, tipe, default=0): |
|---|
| 79 | if ticket.values.has_key(name): |
|---|
| 80 | return tipe(ticket.values[name] or default) |
|---|
| 81 | else: |
|---|
| 82 | cursor = self.env.get_db_cnx().cursor() |
|---|
| 83 | cursor.execute("SELECT * FROM ticket_custom where ticket=%s and name=%s" , (ticket.id, name)) |
|---|
| 84 | val = cursor.fetchone() |
|---|
| 85 | if val: |
|---|
| 86 | return tipe(val[2] or default) |
|---|
| 87 | return default |
|---|
| 88 | |
|---|
| 89 | |
|---|
| 90 | hours = readTicketValue("hours", convertfloat) |
|---|
| 91 | totalHours = readTicketValue("totalhours", convertfloat) |
|---|
| 92 | |
|---|
| 93 | db = self.env.get_db_cnx() |
|---|
| 94 | ticket_id = ticket.id |
|---|
| 95 | cl = ticket.get_changelog() |
|---|
| 96 | |
|---|
| 97 | self.log.debug("found hours: "+str(hours )); |
|---|
| 98 | #self.log.debug("Dir_ticket:"+str(dir(ticket))) |
|---|
| 99 | #self.log.debug("ticket.values:"+str(ticket.values)) |
|---|
| 100 | #self.log.debug("changelog:"+str(cl)) |
|---|
| 101 | |
|---|
| 102 | most_recent_change = None |
|---|
| 103 | if cl: |
|---|
| 104 | most_recent_change = cl[-1]; |
|---|
| 105 | change_time = most_recent_change[0] |
|---|
| 106 | author = most_recent_change[1] |
|---|
| 107 | else: |
|---|
| 108 | change_time = ticket.time_created |
|---|
| 109 | author = ticket.values["reporter"] |
|---|
| 110 | |
|---|
| 111 | |
|---|
| 112 | ## SAVE estimated hour |
|---|
| 113 | estimatedhours = readTicketValue("estimatedhours", convertfloat) |
|---|
| 114 | self.log.debug("found Estimated hours:"+str(estimatedhours)) |
|---|
| 115 | db = self.env.get_db_cnx() |
|---|
| 116 | save_ticket_change( db, ticket_id, author, change_time, "estimatedhours", DONTUPDATE, str(estimatedhours), self.log, True) |
|---|
| 117 | save_custom_field_value( db, ticket.id, "estimatedhours", str(estimatedhours)) |
|---|
| 118 | db.commit(); |
|---|
| 119 | ####################### |
|---|
| 120 | |
|---|
| 121 | |
|---|
| 122 | ## If our hours changed |
|---|
| 123 | if not hours == 0: |
|---|
| 124 | newtotal = str(totalHours+hours) |
|---|
| 125 | save_ticket_change( db, ticket_id, author, change_time, "hours", '0.0', str(hours), self.log) |
|---|
| 126 | save_ticket_change( db, ticket_id, author, change_time, "totalhours", str(totalHours), str(newtotal), self.log) |
|---|
| 127 | save_custom_field_value( db, ticket_id, "hours", '0') |
|---|
| 128 | save_custom_field_value( db, ticket_id, "totalhours", str(newtotal) ) |
|---|
| 129 | ######################## |
|---|
| 130 | |
|---|
| 131 | # END of watch_hours |
|---|
| 132 | |
|---|
| 133 | def ticket_created(self, ticket): |
|---|
| 134 | """Called when a ticket is created.""" |
|---|
| 135 | self.watch_hours(ticket) |
|---|
| 136 | |
|---|
| 137 | |
|---|
| 138 | def ticket_changed(self, ticket, comment, author, old_values): |
|---|
| 139 | """Called when a ticket is modified. |
|---|
| 140 | |
|---|
| 141 | `old_values` is a dictionary containing the previous values of the |
|---|
| 142 | fields that have changed. |
|---|
| 143 | """ |
|---|
| 144 | self.watch_hours(ticket) |
|---|
| 145 | |
|---|
| 146 | def ticket_deleted(self, ticket): |
|---|
| 147 | """Called when a ticket is deleted.""" |
|---|
| 148 | |
|---|
| 149 | |
|---|
| 150 | class TimeTrackingTicketValidator(Component): |
|---|
| 151 | implements(ITicketManipulator) |
|---|
| 152 | |
|---|
| 153 | def __init__(self): |
|---|
| 154 | pass |
|---|
| 155 | |
|---|
| 156 | def prepare_ticket(req, ticket, fields, actions): |
|---|
| 157 | """not currently called""" |
|---|
| 158 | |
|---|
| 159 | def validate_ticket(self, req, ticket): |
|---|
| 160 | """Validate a ticket after it's been populated from user input. |
|---|
| 161 | |
|---|
| 162 | Must return a list of `(field, message)` tuples, one for each problem |
|---|
| 163 | detected. `field` can be `None` to indicate an overall problem with the |
|---|
| 164 | ticket. Therefore, a return value of `[]` means everything is OK.""" |
|---|
| 165 | errors = [] |
|---|
| 166 | #some european countries use , as the decimal separator |
|---|
| 167 | try: |
|---|
| 168 | convertfloat(ticket.values['hours']) |
|---|
| 169 | except KeyError: |
|---|
| 170 | self.log.exception("The hours field was not submitted") |
|---|
| 171 | except ValueError: |
|---|
| 172 | errors.append(('Add Hours to Ticket', 'Value must be a number')) |
|---|
| 173 | try: |
|---|
| 174 | convertfloat(ticket.values['estimatedhours']) |
|---|
| 175 | except KeyError: |
|---|
| 176 | self.log.exception("The estimatedhours field was not submitted") |
|---|
| 177 | except ValueError: |
|---|
| 178 | errors.append(('Estimated Number of Hours', 'Value must be a number')) |
|---|
| 179 | return errors |
|---|