← Back to team overview

mahara-contributors team mailing list archive

[Bug 1520067] Re: Institution blog pagination broken in sites with only 1 institution

 

To replicate:

1. Install Mahara
2. Create a new institution
3. Go to Administration -> Institutions -> Journals
4. Create more than 10 journals for your new institution
5. Again click the "Administration -> Institution -> Journals" link in the menu (so that you again get the starting "institution=1" URL)
6. Try to use pagination to see the rest of your journals

Expected result: You paginate to the second page of journals for the institution
Actual result: Pagination fails, and you see an error message about institution not being entirely alphabetic

This probably got overlooked during testing, because if you have just
created a journal in an institution, then you get returned to the
Journals page with the institution's name in the URL. It's only when you
click the "Administration -> Institution -> Journals" link that you see
the "1" in the URL.

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1520067

Title:
  Institution blog pagination broken when you first visit the
  Institution Journals page

Status in Mahara:
  Confirmed
Status in Mahara 15.10 series:
  Confirmed
Status in Mahara 16.04 series:
  Confirmed

Bug description:
  This patch ( https://reviews.mahara.org/#/c/5784/3 ) fixes institution
  blog pagination. But! When you first go to the Administration ->
  Institutions -> Journals, pagination is still broken. This is because
  on first loading the page, we have the query param "institution=1".
  The artefact/blog/index.php page passes this to
  institution_selector_for_page(), which understands "1" to mean "show
  me the default first institution".

  But, the paginator JS automatically also takes this "1" from the query
  and passes it to the paginator script artefact/blog/index.json.php.
  And that script doesn't know what to do with it, so it crashes and
  says "A required parameter is missing or malformed The 'institution'
  parameter is not alphabetical only".

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1520067/+subscriptions


References