DataError: OFFSET must not be negative

Bug #826839 reported by Diogo Matsubara
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Brad Crittenden
lazr.batchnavigator
Fix Released
Critical
Brad Crittenden

Bug Description

As seen on OOPS-2049AW40 a DataError: OFFSET must not be negative was logged accessing the +translate page.

It seems like an URL hack given that the OOPSes have no referrer. In any case the page shouldn't oops or at least raise a UnexpectedFormData error

Another oops: OOPS-2054H61: ForbiddenAttribute: ('__len__', <storm.sqlobject.SQLObjectResultSet object at 0x1269f150>) was recorded when Brad tried to reproduce this bug.

Tags: oops qa-ok

Related branches

description: updated
Brad Crittenden (bac)
Changed in launchpad:
assignee: nobody → Brad Crittenden (bac)
Brad Crittenden (bac)
Changed in launchpad:
status: Triaged → In Progress
Revision history for this message
Brad Crittenden (bac) wrote :

I've tried to replicate the original problem by hacking the following URL to mimic what I see in the OOPS (note there is no 'memo' field):

https://translations.launchpad.net/transmission/trunk/+pots/transmission/vi/+translate?direction=backwards&start=180

There is no OOPS but the numbering of the items is incorrect as it shows "181 -> 186 of 476" but the first item is 471. See the attached screenshot

Revision history for this message
Brad Crittenden (bac) wrote :

Aha! The OOPS seems to be triggered by attempting to view a translations page with "show=untranslated" and "direction=backwards" when there are no untranslated messages. Probably would happen any time the filter does not match and a direction is given.

Brad Crittenden (bac)
Changed in lazr.batchnavigator:
status: New → Fix Committed
assignee: nobody → Brad Crittenden (bac)
importance: Undecided → Critical
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Revision history for this message
Brad Crittenden (bac) wrote :

https://translations.qastaging.launchpad.net/transmission/trunk/+pots/transmission/eu/+translate?start=0&batch=10&show=untranslated&direction=backwards

The Basque (eu) translation on qastaging has zero untranslated strings. Using the above URL to replicate the problem of going backwards in an empty result set did not OOPS. The problem is fixed.

tags: added: qa-ok
removed: qa-needstesting
Gary Poster (gary)
Changed in launchpad:
status: In Progress → Fix Released
Brad Crittenden (bac)
Changed in lazr.batchnavigator:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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