Launchpad itself

Oops at /projects/+syncreview

Bug #750 reported by Matthew Paul Thomas
6
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Medium
David Allouche

Bug Description

https://launchpad.ubuntu.com/projects/+syncreview returns an error:
/srv/launchpad.net/production/launchpad/lib/canonical/launchpad/templates/bazaar-sync-review.pt
Line 49, Column 20
Expression: <PathExpr standard:'series/needsSyncReview'>
...
    * __traceback_info__: (<ProductSeries at 0x61e5890>, 'needsSyncReview', [])
NotFoundError: (<ProductSeries at 0x61e5890>, 'needsSyncReview')

OOPS-310D743

See original description
Morgan Collett (morgan)
Changed in launchpad:
assignee: nobody → morgan
Revision history for this message
Morgan Collett (morgan) wrote : The error

NotFoundError: (<ProductSeries at 0x843aaf6c>, 'needsSyncReview')

Revision history for this message
Christian Reis (kiko) wrote : Re: /projects/+syncreview causes System Error

    * Module zope.pagetemplate.pagetemplate, line 114, in pt_render
      raise PTRuntimeError(str(self._v_errors))
      Warning: Compilation failed
      Warning: zope.tal.taldefs.TALError: Invalid variable name "product products" in expression 'product products', at line 29, column 13

PTRuntimeError: ['Compilation failed', 'zope.tal.taldefs.TALError: Invalid variable name "product products" in expression \'product products\', at line 29, column 13']

Changed in launchpad:
assignee: morgan → kiko
Christian Reis (kiko)
description: updated
summary: - Steps to reproduce:
- 1
Revision history for this message
Christian Reis (kiko) wrote :

NotFoundError: (<ProductSeries at 0xede8f90>, 'needsSyncReview')

Sigh.

Christian Reis (kiko)
Changed in launchpad:
assignee: kiko → diogomatsubara
Matthew Paul Thomas (mpt)
description: updated
Revision history for this message
Diogo Matsubara (matsubara) wrote :

What happened to method needsSyncReview? Was it substituted by syncCertified?

Diogo Matsubara (matsubara)
Changed in launchpad:
status: New → NeedInfo
Matthew Paul Thomas (mpt)
Changed in launchpad:
status: Needs Info → Confirmed
Revision history for this message
David Allouche (ddaa) wrote :

I have no freakin' memory what this page was meant to do, and from the look of it it appears superceded by /bazaar/series.

My guess would be that needsSyncReview was removed when SourceSource was moved into ProductSeries (or some time before that).

As far as I'm concerned, feel free to remove it.

Diogo Matsubara (matsubara)
description: updated
Revision history for this message
Diogo Matsubara (matsubara) wrote :

David removed that page in r4295

Changed in launchpad:
assignee: matsubara → ddaa
status: Confirmed → Fix Committed
Matthew Paul Thomas (mpt)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.