- Issue created by @kopeboy
- First commit to issue fork.
- Merge request !2#3360808: Fatal error: Access level to Drupal\config_view\Plugin\views\query\ConfigViewQuery::$offset must be public → (Open) created by wjackson
- Status changed to Needs review
about 1 year ago 9:24pm 24 January 2024 - 🇺🇸United States wjackson Aiken, SC
Attached is the .patch file referenced in the merge request referenced in the previous comment. Seems to resolve the issue with how I use the module, but let us know if any additional changes are needed.
Thank you!
- Status changed to RTBC
about 1 year ago 1:42pm 31 January 2024 - 🇮🇹Italy kopeboy Milan
Thanks, that fixed the issue.
I learnt about the merge with config_views module, I will post new issues there.
- Status changed to Needs review
5 months ago 8:06pm 13 September 2024 - 🇺🇸United States edwardsay
The module does not work on 10.3:
Fatal error: Access level to Drupal\config_view\Plugin\views\query\ConfigViewQuery::$offset must be public (as in class Drupal\views\Plugin\views\query\QueryPluginBase) in /app/web/modules/contrib/config_view/src/Plugin/views/query/ConfigViewQuery.php on line 23"
Attached is the patch that resolves this issue - 🇬🇧United Kingdom scott_euser
scott_euser → changed the visibility of the branch 3360808-fatal-error-access to hidden.
- 🇬🇧United Kingdom scott_euser
scott_euser → changed the visibility of the branch 9.0.x to hidden.
- Merge request !4Update ConfigViewQuery to include offset with default to resolve errors when used. → (Open) created by scott_euser
- 🇬🇧United Kingdom scott_euser
Just re-applying the patch, but confirming this is required for the module to work. There is no workaround, so I think given that Critical is probably right status.
Steps I tried:
- Enable Role configuration
- Go to add new View
- Select Role
- Fatal error without patch. Success with patch