Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-12186

Oracle SQL Statement problem -- unable to list glossary entries

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8.2
    • Fix Version/s: 1.8.4, 1.9, 2.0
    • Component/s: Glossary
    • Labels:
      None
    • Environment:
      Red Hat Enterprise Linux ES release 4 (Nahant Update 5)
      PHP Version 5.2.3
      Oracle Database 10g Release 10.2.0.1.0
    • Database:
      Oracle
    • URL:
      sorry, intranet :)
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      While trying to use the auto-linking functionality with the glossary, I got nothing.....
      When looking into code, I found a bug ; it seems to be relative to Oracle
      The clause: != \'\' appears to not working under my Oracle....???
      And obviously is not null does (but does not test the same).

      Another problem shows up after it, when having aliases without entries or categories, the array_merge with a null array fails !

      I've modified this in the joined file. Here's the diff lines:
      73,74c73,74
      < AND ge.concept != \'\' ');
      <

      > AND ge.concept is not null '); //Joseph Boiteau: previous sql isn't working with Oracle
      > // and moreover, the string could'nt be emptyl using interface
      85c85,86
      < if ($aliases) {

      > // Joseph Boiteau: adding controls to not use a null $concepts object
      > if ($aliases and $concepts) {
      87a89,91
      > elseif ($aliases)

      { > $concepts = $aliases; > }

      Should I commit that somewhere ?
      Is it a bad sql or an Oracle abnormal comportment ?

      It's working here but I don't know what to do with that !

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                stronk7 Eloy Lafuente (stronk7)
                Reporter:
                jobano Joseph Boiteau
                Tester:
                Nobody
                Participants:
                Component watchers:
                Adrian Greeve, Mihail Geshoski, Peter Dias
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Jan/08