- First commit to issue fork.
- Merge request !36Issue 3275926: Google_analytics TypeError issue resolved. → (Merged) created by Sapnakhatri
- last update
about 1 year ago 15 pass - last update
about 1 year ago 15 pass - Status changed to Needs review
about 1 year ago 7:47am 15 February 2024 - 🇮🇳India Sapnakhatri
I have added is_null() to check the account is null or not.Resolved this issue.
- 🇨🇦Canada mahde Vancouver
Thanks @Sapnakhatri - Your update resolved the issue!
- Status changed to RTBC
10 months ago 11:43am 9 May 2024 - 🇮🇳India manish-31
I have reviewed and tested this patch https://git.drupalcode.org/project/google_analytics/-/merge_requests/36...., applies successfully and fixes the reported error.
To replicate the error I have set the value of "account" in
google_analytics.setting
config to NULL. - First commit to issue fork.
- 🇺🇸United States japerry KVUO
Comment #14 hides the actual issue, which is that account info is null but should be an empty string. By properly type setting the variables, these issues won't occur in the first place.
- 🇺🇸United States japerry KVUO
Oh it also fixed the config issue I was seeing in Drupal 11.2. Committed!
-
japerry →
committed 0c0e0691 on 4.x authored by
sapnakhatri →
Issue #3275926 by sapnakhatri: Fix TypeError with accounts config...
-
japerry →
committed 0c0e0691 on 4.x authored by
sapnakhatri →
Automatically closed - issue fixed for 2 weeks with no activity.