Skip to content

Commit

Permalink
Merge branch 'wip_MDL-49631_28_utfregex' of https://github.com/skodak…
Browse files Browse the repository at this point in the history
…/moodle into MOODLE_28_STABLE
  • Loading branch information
danpoltawski committed Mar 24, 2015
2 parents 9e84f90 + 4f431ff commit f7751d8
Show file tree
Hide file tree
Showing 3 changed files with 50 additions and 3 deletions.
4 changes: 2 additions & 2 deletions calendar/lib.php
Original file line number Diff line number Diff line change
Expand Up @@ -2944,7 +2944,7 @@ function calendar_add_icalendar_event($event, $courseid, $subscriptionid, $timez
$name = $event->properties['SUMMARY'][0]->value;
$name = str_replace('\n', '<br />', $name);
$name = str_replace('\\', '', $name);
$name = preg_replace('/\s+/', ' ', $name);
$name = preg_replace('/\s+/u', ' ', $name);

$eventrecord = new stdClass;
$eventrecord->name = clean_param($name, PARAM_NOTAGS);
Expand All @@ -2956,7 +2956,7 @@ function calendar_add_icalendar_event($event, $courseid, $subscriptionid, $timez
$description = clean_param($description, PARAM_NOTAGS);
$description = str_replace('\n', '<br />', $description);
$description = str_replace('\\', '', $description);
$description = preg_replace('/\s+/', ' ', $description);
$description = preg_replace('/\s+/u', ' ', $description);
}
$eventrecord->description = $description;

Expand Down
2 changes: 1 addition & 1 deletion lib/moodlelib.php
Original file line number Diff line number Diff line change
Expand Up @@ -1102,7 +1102,7 @@ function clean_param($param, $type) {
// Remove some nasties.
$param = preg_replace('~[[:cntrl:]]|[<>`]~u', '', $param);
// Convert many whitespace chars into one.
$param = preg_replace('/\s+/', ' ', $param);
$param = preg_replace('/\s+/u', ' ', $param);
$param = core_text::substr(trim($param), 0, TAG_MAX_LENGTH);
return $param;

Expand Down
47 changes: 47 additions & 0 deletions lib/tests/regex_test.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,47 @@
<?php
// This file is part of Moodle - http://moodle.org/
//
// Moodle is free software: you can redistribute it and/or modify
// it under the terms of the GNU General Public License as published by
// the Free Software Foundation, either version 3 of the License, or
// (at your option) any later version.
//
// Moodle is distributed in the hope that it will be useful,
// but WITHOUT ANY WARRANTY; without even the implied warranty of
// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
// GNU General Public License for more details.
//
// You should have received a copy of the GNU General Public License
// along with Moodle. If not, see <http://www.gnu.org/licenses/>.

/**
* Test PHP regex capability - this may also serve as an example for devs.
*
* @package core
* @copyright 2015 Totara Learning Solutions Ltd {@link http://www.totaralms.com/}
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @author Petr Skoda <petr.skoda@totaralms.com>
*/

defined('MOODLE_INTERNAL') || die();

/**
* Test PHP regex capability - this may also serve as an example for devs.
*
* @package core
* @copyright 2015 Totara Learning Solutions Ltd {@link http://www.totaralms.com/}
* @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
* @author Petr Skoda <petr.skoda@totaralms.com>
*/
class core_regex_testcase extends advanced_testcase {
public function test_whitespace_replacement_with_u() {
$unicode = "Теорія і практика використання системи управління навчанням Moo
dleКиївський національний університет будівництва і архітектури, 21-22 тра
вня 2015 р.http://2015.moodlemoot.in.ua/";

$whitespaced = preg_replace('/\s+/u', ' ', $unicode);
$this->assertSame(str_replace("\n", ' ', $unicode), $whitespaced);
}
}


0 comments on commit f7751d8

Please sign in to comment.