Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#128 closed defect (fixed)

Several tickets fail to display now (due to SensitiveTickets plugin bug)

Reported by: andersk Owned by:
Priority: blocker Milestone:
Component: misc Keywords:
Cc:

Description

Tickets #1 and #2 (at least) in this Trac are broken. We should fix them.

scripts / trac-#127 / adehnert  22:36  (Alex Dehnert)
    adehnert filed a new ticket:
    ttants: 1, 687224

    http://scripts.mit.edu/trac/ should be aware that ticket number 1 is
    not the same as number 687224.
scripts / trac-not-#127 / andersk  01:20  (Anders Kaseorg)
    Huh.
    http://www.google.com/search?q=ticket+687224
    http://www.google.com/search?q=ticket+433633
scripts / trac-not-#127 / andersk  01:21  (Anders Kaseorg)
    Oh, I bet it complains that ticket 687224 doesn’t exist because it’s
    trying to linkify the text “[help.mit.edu #687224]” in the comment.
scripts / trac-not-#127 / quentin  01:22  (Quentin Smith)
    Oh. It's probably trying to check the permissions on that link...
scripts / trac-not-#127 / quentin  01:22  (Quentin Smith)
    (I didn't disable that part of the sensitivetickets plugin...)
scripts / trac-not-#127 / andersk  01:22  (Anders Kaseorg)
    Why… does it care about the permissions on that link?
scripts / trac-not-#127 / quentin  01:23  (Quentin Smith)
    Presumably because it's trying to find out the title and state of the
    tickets so it can put a TITLE on the link?

Attachments (1)

sensitivetickets-deny-nonexistent.patch (2.1 KB) - added by andersk 12 years ago.

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by andersk

comment:1 Changed 12 years ago by andersk

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

Fixed by the attached patch. I’ll send it upstream.

Note: See TracTickets for help on using tickets.