- 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
over 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
over 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
10 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
- Status changed to RTBC
8 days ago 10:33am 28 June 2025 Not sure why you're hiding patches since the issue is still not solved yet. Here's one that worked for me.
- 🇬🇧United Kingdom scott_euser
The merge request is for the ongoing development, the advice in the documentation is to either download the patch or use composer patches 2. Patches make it harder for maintainers to know what to review + worst case even diverge from the merge request causing people to waste efforts. https://www.drupal.org/docs/develop/git/using-gitlab-to-contribute-to-dr... →
Thanks!