mahara-contributors team mailing list archive

[Aaron Wells <1215702@xxxxxxxxxxxxxxxxxx>]

** Changed in: mahara
    Milestone: 1.8rc1 => 1.8.0

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

Title:
  Reduce false positives in syntax checker for unbracketed SQL tables

Status in Mahara ePortfolio:
  Fix Committed
Status in Mahara 1.5 series:
  Fix Committed
Status in Mahara 1.6 series:
  Fix Committed
Status in Mahara 1.7 series:
  Fix Committed

Bug description:
  The check for SQL tables that aren't surrounded by {}, which was added
  to test/coding-standard-check.pl, also matches comments that contain
  these very common words, and that results in a lot of false positives
  from the script, which mkaes people less likely to pay attention to
  its output.

  We should reduce the false positives by changing the regular
  expression so that it ignores the words "from" and "join" when they
  occur in comments.

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