Problem/Motivation

I seem to have found some cases where the number of search results doesn't match the number displayed to the end user. This mucks up things like the pager.

The module schema specifies general_settings.results as the results value, so let's standardise on that.

Steps to reproduce

  1. Set general_settings.pagesize to 20
  2. Leave general_settings.results at 10
  3. Perform any search

Proposed resolution

Deprecate general_settings.pagesize and use general_settings.results for all searching and pagination operations.

Remaining tasks

Implement the change.

User interface changes

None.

API changes

None.

Data model changes

Ensure general_settings.pagesize is removed from the module entirely. Since it's not in the schema this should work out OK.

Issue fork funnelback-3435248

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

geoffreyr created an issue. See original summary.

geoffreyr’s picture

Assigned: geoffreyr » Unassigned
Status: Active » Needs review
larowlan’s picture

Status: Needs review » Needs work

Do we need a post-update hook to convert over any config that is using the old (wrong) key?

geoffreyr’s picture

Version: 2.0.0-beta2 » 2.0.0-beta3
Assigned: Unassigned » geoffreyr
Status: Needs work » Needs review

Updated MR with a post-update hook, as well as a module schema file.

geoffreyr’s picture

Assigned: geoffreyr » Unassigned