Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.0.2
    • Component/s: Libraries
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      The new static method introduced in MDL-25886 does not handle NULL values correctly because isset() is used there. NULL values are allowed - for example for "imagealt" field.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            mudrd8mz David Mudrák added a comment -

            Replaced isset() with property_exists() and added a unit test for this bug.

            Show
            mudrd8mz David Mudrák added a comment - Replaced isset() with property_exists() and added a unit test for this bug.
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Sorry David hit the wrong button

            Show
            samhemelryk Sam Hemelryk added a comment - Sorry David hit the wrong button
            Hide
            tsala Helen Foster added a comment -

            Thanks David for reporting and fixing this issue

            Show
            tsala Helen Foster added a comment - Thanks David for reporting and fixing this issue

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  21/Feb/11