From: stronk7 Date: Thu, 22 Nov 2007 22:47:49 +0000 (+0000) Subject: Deleting one ='' condition that isn't cross-db nor necessary. MDL-12186 X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=5fe600ebe745fe87cd9c73f1d3e4ef1d46f4b180;p=moodle.git Deleting one ='' condition that isn't cross-db nor necessary. MDL-12186 Merged from MOODLE_19_STABLE --- diff --git a/mod/glossary/filter.php b/mod/glossary/filter.php index 90ec18b38b..ba4525a440 100644 --- a/mod/glossary/filter.php +++ b/mod/glossary/filter.php @@ -52,28 +52,26 @@ function glossary_filter($courseid, $text) { /// Pull out all the raw data from the database for entries, categories and aliases - $entries = get_records_select('glossary_entries', - 'glossaryid IN ('.$glossarylist.') AND usedynalink != 0 AND '. - 'approved != 0 AND concept != \'\'','', - 'id,glossaryid, concept,casesensitive,0 AS category,fullmatch'); + $entries = get_records_select('glossary_entries', + 'glossaryid IN ('.$glossarylist.') AND usedynalink != 0 AND approved != 0 ', '', + 'id,glossaryid, concept, casesensitive, 0 AS category, fullmatch'); - $categories = get_records_select('glossary_categories', + $categories = get_records_select('glossary_categories', 'glossaryid IN ('.$glossarylist.') AND usedynalink != 0', '', - 'id,glossaryid,name AS concept, '. - '1 AS casesensitive, 1 AS category, 1 AS fullmatch'); - - $aliases = get_records_sql('SELECT ga.id, ge.glossaryid, ga.alias as concept, ge.concept as originalconcept, - casesensitive, 0 AS category, fullmatch - FROM '.$CFG->prefix.'glossary_alias ga, + 'id,glossaryid,name AS concept, 1 AS casesensitive, 1 AS category, 1 AS fullmatch'); + + $aliases = get_records_sql('SELECT ga.id, ge.glossaryid, ga.alias as concept, ge.concept as originalconcept, + casesensitive, 0 AS category, fullmatch + FROM '.$CFG->prefix.'glossary_alias ga, '.$CFG->prefix.'glossary_entries ge WHERE ga.entryid = ge.id AND ge.glossaryid IN ('.$glossarylist.') - AND ge.usedynalink != 0 - AND ge.approved != 0 - AND ge.concept != \'\' '); + AND ge.usedynalink != 0 + AND ge.approved != 0'); /// Combine them into one big list + $concepts = array(); if ($entries and $categories) { $concepts = array_merge($entries, $categories); } else if ($categories) { @@ -81,11 +79,11 @@ function glossary_filter($courseid, $text) { } else if ($entries) { $concepts = $entries; } - + if ($aliases) { $concepts = array_merge($concepts, $aliases); } - + if (!empty($concepts)) { foreach ($concepts as $key => $concept) { /// Trim empty or unlinkable concepts