- Issue created by @kalpaitch
- last update
about 1 year ago 12 pass - @kalpaitch opened merge request.
Since Pagerer 8.x-2.0 it is possible to override Drupal core's querystring so to use dots instead of commas
My pagerer.settings.yml does not have the `url_querystring` parameters, I have no idea why the settings were not exported following the `pagerer_update_8211`
Either way , when upgrading to Pagerer 3.0, which has stricter type checking, I get the following error:
TypeError: Cannot assign null to property Drupal\pagerer\PagererParameters::$querystringOverride of type bool in Drupal\pagerer\PagererParameters->__construct()
The constructor for PagererParameters and PagererManager both assume `url_querystring.core_override` contains a valid boolean value and do not provide a default.
Now I realise everything has been done to make sure these `url_querystring` settings are present on all installations, but in these is there a reason the constructors for PagererParameters and PagererManager couldn't set a default value for `core_override` (which they do for some of the other `url_querystring` properties)?? Anyway I'm putting this patch in to temporarily resolve the issue for myself while I deploy the updated settings.
Install version 2.0 of the module
Remove all `url_querystring` settings from the site and re-import the config
Upgrade to version 3.0
Active
3.0
Code