mahara-contributors team mailing list archive

[Aaron Wells <1484296@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

A client reported this issue to me while testing the patch
https://reviews.mahara.org/5062

They found that after that patch was in place ALL leap import/export
operations produced this error:

"[WAR] export/leap/lib.php:161 preg_replace(): Compilation failed: non-
hex character in \x{} (closing brace missing?) at offset 92"

On inspection in the code, it's obvious that there is indeed a syntax
error in the regex that I wrote in patch
https://reviews.mahara.org/5061. Specifically a missing closing brace.
It says "\x{FDEO" where it should be "\x{FDEO}". Strangely the presence
of this syntax error did not cause any problems when running this code
on my local machine. I suspect it may be a difference in the version of
the PCRE library that we're running.

** Affects: mahara
     Importance: High
     Assignee: Aaron Wells (u-aaronw)
         Status: In Progress

** Affects: mahara/15.04
     Importance: High
     Assignee: Aaron Wells (u-aaronw)
         Status: In Progress


** Tags: export import leap2a

** Also affects: mahara/15.04
   Importance: Undecided
       Status: New

** Changed in: mahara/15.04
     Assignee: (unassigned) => Aaron Wells (u-aaronw)

** Changed in: mahara/15.04
    Milestone: None => 15.04.3

** Changed in: mahara/15.04
   Importance: Undecided => High

** Changed in: mahara/15.04
       Status: New => In Progress

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

Title:
  Typo in XML filter regex

Status in Mahara:
  In Progress
Status in Mahara 15.04 series:
  In Progress

Bug description:
  A client reported this issue to me while testing the patch
  https://reviews.mahara.org/5062

  They found that after that patch was in place ALL leap import/export
  operations produced this error:

  "[WAR] export/leap/lib.php:161 preg_replace(): Compilation failed:
  non-hex character in \x{} (closing brace missing?) at offset 92"

  On inspection in the code, it's obvious that there is indeed a syntax
  error in the regex that I wrote in patch
  https://reviews.mahara.org/5061. Specifically a missing closing brace.
  It says "\x{FDEO" where it should be "\x{FDEO}". Strangely the
  presence of this syntax error did not cause any problems when running
  this code on my local machine. I suspect it may be a difference in the
  version of the PCRE library that we're running.

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