InvalidBatchSizeErrorView reports configured maximum batch size but this can be overridden.

Bug #400597 reported by Henning Eggers
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Henning Eggers

Bug Description

For example, POFileTranslateView uses POFileBatchNavigator which overrides the max_batch_size property of BatchNavigator to enforce a lower value. The ErrorView should use that same property instead of getting it from the config directly.

Revision history for this message
Henning Eggers (henninge) wrote :

Really, lazr.batchnavigator should propagate the maximum value in the exception but it only returns an error message which contains this value. So without changing anything in lazr, I'd propose to display that error message. Here is a fix that would do that, I can land that today.

https://pastebin.canonical.com/19941/

Revision history for this message
Henning Eggers (henninge) wrote :

The error message from lazr is:
Maximum for "batch" parameter is 300.

Changed in launchpad-foundations:
assignee: nobody → Henning Eggers (henninge)
importance: Undecided → High
milestone: none → 2.2.7
status: New → In Progress
Revision history for this message
Diogo Matsubara (matsubara) wrote : Bug fixed by a commit

Fixed in devel r8948.

Changed in launchpad-foundations:
status: In Progress → Fix Committed
Revision history for this message
Henning Eggers (henninge) wrote :

Fixed in db-devel r8290.

Changed in launchpad-foundations:
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.