← Back to branch summary

~launchpad-pqm/launchpad/devel

« back to all changes in this revision

Viewing changes to lib/lp/bugs/browser/bugtask.py

[r=sinzui][bug=904890] When an attempt is made to delete a
        non-existent bugtask,
        inform the user and delete the affected table row in the ui.

Show diffs side-by-side

added added

removed removed

Lines of Context:
lib/lp/bugs/browser/bugtask.py
559
559
                # Security proxy this object on the way out.
560
560
                return getUtility(IBugTaskSet).get(bugtask.id)
561
561
 
562
 
        # If we've come this far, there's no task for the requested
563
 
        # context. Redirect to one that exists.
 
562
        # If we've come this far, there's no task for the requested context.
 
563
        # If we are attempting to navigate past the non-existent bugtask,
 
564
        # we raise NotFound error. eg +delete or +edit etc.
 
565
        # Otherwise we are simply navigating to a non-existent task and so we
 
566
        # redirect to one that exists.
 
567
        travseral_stack = self.request.getTraversalStack()
 
568
        if len(travseral_stack) > 0:
 
569
            raise NotFoundError
564
570
        return self.redirectSubTree(canonical_url(bug.default_bugtask))
565
571
 
566
572
 
2253
2259
            }
2254
2260
 
2255
2261
        # This is a total hack, but pystache will run both truth/false values
2256
 
        # for an empty list for some reason, and it "works" if it's just a flag
2257
 
        # like this. We need this value for the mustache template to be able
2258
 
        # to tell that there are no tags without looking at the list.
 
2262
        # for an empty list for some reason, and it "works" if it's just a
 
2263
        # flag like this. We need this value for the mustache template to be
 
2264
        # able to tell that there are no tags without looking at the list.
2259
2265
        flattened['has_tags'] = True if len(flattened['tags']) else False
2260
2266
        return flattened
2261
2267