mahara-contributors team mailing list archive

[Nigel McNie <nigel@xxxxxxxxxx>]

Public bug reported:

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.

** Affects: mahara
     Importance: Undecided
         Status: New


** Tags: import leap2a

-- 
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:
  New

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