Not able to authorise account after entering all details. It is redirecting to home page instead of google analytics api page. I have checked the watchdog for errors, it displays "Page not found" with google referrer url.

The issue is because of the incorrect Authorized redirect url displayed in API page. Attached image.
error

This only happens if there is multi language and all language has Prefix code.

CommentFileSizeAuthor
#5 3002754-#3-google-analytics-report-fix-for-multilanguage_v2.patch2.58 KBsathish.redcrackle
#4 3002754-#3-google-analytics-report-fix-for-multilanguage.patch2.72 KBsathish.redcrackle
#2 google analytics report error.png306.78 KBsathish.redcrackle
google analytics report error.png306.78 KBsathish.redcrackle
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sathish.redcrackle created an issue. See original summary.

sathish.redcrackle’s picture

sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commented
Issue summary: View changes
FileSize
google analytics report error.png306.78 KB
sathish.redcrackle’s picture

sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commented
Issue summary: View changes
sathish.redcrackle’s picture

sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commented
FileSize
3002754-#3-google-analytics-report-fix-for-multilanguage.patch2.72 KB

"request_path()" is the function used to get the current path and passing that value as redirect url to google analytics.
I have changed the "request_path()" to "current_path()"

Attached the patch.

sathish.redcrackle’s picture

sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commented
FileSize
3002754-#3-google-analytics-report-fix-for-multilanguage_v2.patch2.58 KB

Removed the drupal module path and updated the patch file.

sathish.redcrackle’s picture

sathish.redcrackle CreditAttribution: sathish.redcrackle at Red Crackle commented
Status: Active » Needs review

  • klausi committed 47fc0632 on 7.x-3.x 
    fix(oauth): Not able to authorise account if multi language is enabled...
klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi at jobiqo - job board technology commented
Version: 7.x-3.1 » 7.x-3.x-dev
Status: Needs review » Fixed

Patch does not apply because of the "google_analytics_reports" prefix, but managed to apply it manually. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.