Reviewed:  https://reviews.mahara.org/8037
Committed: 
https://git.mahara.org/mahara/mahara/commit/943ce498fb9884be0f66343a35e8a8924d5d4fb5
Submitter: Robert Lyon ([email protected])
Branch:    master

commit 943ce498fb9884be0f66343a35e8a8924d5d4fb5
Author: Robert Lyon <[email protected]>
Date:   Wed Sep 20 11:38:16 2017 +1200

Bug 1513276: Fixing iframe source mistake

And adding info to help developers

behatnotneeded

Change-Id: Idebbf277cd37f4406874bddb66b4ffef71088750
Signed-off-by: Robert Lyon <[email protected]>

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask 
on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1513276

Title:
  Consolidate the iframe sources code

Status in Mahara:
  Fix Committed

Bug description:
  Currently we do a bunch of different things to add allowed iframe
  sources code to the system when we install/upgrade a site

  We need to make it one way only

  Notes:
  lib/upgrade.php, core_postinst(), has a hard-coded list of allowed iframe 
sources
  if you add the youtube-no-iframes one there, can you also move the prezi one 
there, and take it out from blocktype/externalvideo::postinst() ?

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1513276/+subscriptions

_______________________________________________
Mailing list: https://launchpad.net/~mahara-contributors
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~mahara-contributors
More help   : https://help.launchpad.net/ListHelp

Reply via email to