Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.4, 2.2.1, 2.3
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: General
    • Labels:

      Description

      1/ concentrate on false returned from get_config() when plugin does not have any entries yet - it is a bit similar to old get_records()

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            to integrators: I have carefully reviewed all places that use get_config('componentname') and it does not seem there could be any regressions caused by the NULL-->stdClass change

            Show
            skodak Petr Skoda added a comment - to integrators: I have carefully reviewed all places that use get_config('componentname') and it does not seem there could be any regressions caused by the NULL-->stdClass change
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (21, 22 & master). Thanks! I've been performing some searches and have not been able to find more uses dependent of the change.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (21, 22 & master). Thanks! I've been performing some searches and have not been able to find more uses dependent of the change.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            About to run install & upgrade in all versions, will look for any notice/warn/error.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - About to run install & upgrade in all versions, will look for any notice/warn/error.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Passing, I've reviewed all install and upgrade combinations between 2.x versions and none throws notices/warns.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Passing, I've reviewed all install and upgrade combinations between 2.x versions and none throws notices/warns.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Your nice code represents only 1/46 of the issues that have been sent upstream this week, so thanks, but not many.

            Nah, joking, many thanks! Closing this a fixed, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Your nice code represents only 1/46 of the issues that have been sent upstream this week, so thanks, but not many. Nah, joking, many thanks! Closing this a fixed, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  12/Mar/12