mahara-contributors team mailing list archive

[Mahara Bot <1379060@xxxxxxxxxxxxxxxxxx>]

Patch for "1.10_STABLE" branch: https://reviews.mahara.org/3830

-- 
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/1379060

Title:
  error message shows span tag escaped

Status in Mahara ePortfolio:
  Fix Committed

Bug description:
  For example if I add a external feed block to a page and then add in
  gibberish for the url I get this error on save

  <span id="instconf_url_error">The feed appears to be invalid. The
  error reported was: Could not resolve host: dfd</span>

  but the problem is it's displaying the <span> as escaped html.

  This patch https://reviews.mahara.org/#/c/3731/3 deals with sorting out the basic rules from being escaped - but there are other cases where the code has specified their specific error mesaages, eg 
  $form->set_error('url', get_string('invalidfeed', 'blocktype.externalfeed',  $e->getMessage()));

  needs to have a third param false added to it and the $error message
  escaped

  $form->set_error('url', get_string('invalidfeed',
  'blocktype.externalfeed',  hsc($e->getMessage())), false);

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