Comment 4 for bug 1353759

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/3657
Committed: http://gitorious.org/mahara/mahara/commit/dfc0711bb14645732e90c997e747b7ba1cb38b65
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.8_STABLE

commit dfc0711bb14645732e90c997e747b7ba1cb38b65
Author: Robert Lyon <email address hidden>
Date: Fri Aug 8 12:18:41 2014 +1200

More Google maps urls not working in Google Apps block (Bug #1353759)

Was not working for urls with @ latitutude,longitude, zoom eg:
https://www.google.com/maps/@37.509345,-121.9596665,12z

And not working for shorthand ulrs like https://goo.gl/maps/pZTiA

So I've added a check for the urls with @ latitutude,longitude, zoom
and turned them into an acceptible embed url. And for shorthand urls
I've added a generic function to fetch the url the short one resolves
to and so it can parse that one instead of the short url.

Change-Id: I3c76d2608e9708f1722e243cdc02a43f1885b815
Signed-off-by: Robert Lyon <email address hidden>