mahara-contributors team mailing list archive

[Son Nguyen <984575@xxxxxxxxxxxxxxxxxx>]

I have done several tests for this patch. However, it did not work as expected.
Some child HTML tags of <content> tag did not import such as <p>, <a>, <img>, <table>, <ol>, and <ul>.

See my import file in the attached file?field.comment=I have done several tests for this patch. However, it did not work as expected.
Some child HTML tags of <content> tag did not import such as <p>, <a>, <img>, <table>, <ol>, and <ul>.

See my import file in the attached file

** Attachment added: "importLeap2A.zip"
   https://bugs.launchpad.net/mahara/+bug/984575/+attachment/3308334/+files/importLeap2A.zip

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/984575

Title:
  Leap2A importer: improve robustness of <content> importing

Status in Mahara ePortfolio:
  In Progress

Bug description:
  In import/leap/lib.php, function fix_artefact_reference.

  If a <content> tag in a Leap2A import has more than one child node,
  the importer gets confused and only imports one of them (either the
  first or the last, not sure which).

  E.g.:

          <content type="html">&lt;p&gt;&amp;nbsp;&lt;/p&gt; &lt;p&gt;a
  paragraph&lt;/p&gt; &lt;p&gt;&amp;nbsp;&lt;/p&gt</content>

  This is:

  <content>
    <p>
    <p>
    <p>

  Which isn't handled properly. I think the leap2A spec mentions that
  content like this is not a good idea, but the fix seems pretty easy.
  Patch attached.

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