268801 - Next links don't work properly when showing 50 items per page

Status: RESOLVED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Andy Roberts]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.5) Gecko/20041107 Firefox/1.0

When updating the list of extensions to show 50 items per page, the page is
displayed correctly however, clicking next will cause the second page to revert
to showing 10 per page, i.e. items 11 to 20.

Reproducible: Always
Steps to Reproduce:
1. Click browse extensions
2. Click All category
3. Select 50 from the Show/Page dropdown
4. Click update
5. Click the next link

Actual Results:  
Next page contains extensions 11 to 20, page numbers revert to 2 of 8.

Expected Results:  
Next page should contain extensions 51 to xx (77 at time of writing), and page
number should be 2 of 2.

Comment 1

[alanjstr]

Do you have cookies enabled?

Comment 2

[Andy Roberts]

Yes, all settings are as per default Firefox 1.0 - I'd just installed it.

Comment 3

[Ian Butcher]

In addition, if you select a different version of Firefox from the default (I
selected 1.0PR) on each subsequent page, the view returns to the default.

For the default view, when browsing all - there are 77 (reported) extensions,
switch to 1.0PR, and there are 177 reported extensions - however, after page 2
(or next) is selected, the same problem occurs - ie I get 11-20 of 77 displayed.

Comment 4

[chris_shaffer]

Specifically, the numpage variable is not passed.  The link should be:

https://update.mozilla.org/extensions/showlist.php?application=firefox&category=&numpg=50&pageid=2

and instead, it is:

https://update.mozilla.org/extensions/showlist.php?application=firefox&category=&numpg=&pageid=2

Comment 5

[Wolf [:wolf]]

Fallout from, Bug 268781.

Comment 6

[Wolf [:wolf]]

Attachment: Fix numpg variable and Add version variable.

A fix. :-) yay.

Comment 7

[Wolf [:wolf]]

Checked in. 

Comment 8

[Wolf [:wolf]]

*** Bug 268943 has been marked as a duplicate of this bug. ***