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

XMLDB generated upgrade code gives warnings in codechecker.

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Database SQL/XMLDB
    • Labels:
    • Testing Instructions:
      Hide
      1. Open the XMLDB Editor
      2. Generate each possible PHP code
        • Edit a file
          1. Click on 'View PHP code'
          2. For each element in the action dropdown menu: make sure no errors appear and the coding guidelines are respected
        • Click 'Back'
        • Select a table
          1. Click on 'View PHP code'
          2. For each element in the action dropdown menu: make sure no errors appear and the coding guidelines are respected
      Show
      Open the XMLDB Editor Generate each possible PHP code Edit a file Click on 'View PHP code' For each element in the action dropdown menu: make sure no errors appear and the coding guidelines are respected Click 'Back' Select a table Click on 'View PHP code' For each element in the action dropdown menu: make sure no errors appear and the coding guidelines are respected
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-37686-master

      Description

      Does not obey max line length or comment formatting (end in full stop).

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            skodak Petr Skoda added a comment -

            Thanks for the report.

            Show
            skodak Petr Skoda added a comment - Thanks for the report.
            Hide
            andyjdavis Andrew Davis added a comment -

            [Y] Syntax
            [Y] Output
            [?] Whitespace
            [Y] Language
            [NA] Databases
            [?] Testing
            [NA] Security
            [NA] Documentation
            [Y] Git
            [Y] Sanity check

            From the testing instructions.
            "Make sure it works and the coding guidelines are respected"
            How would one go about doing that? I feel like you're missing a step here.

            You didn't introduce it but there is a space missing on line 522 of admin/tool/xmldb/actions/view_table_php/view_table_php.class.php
            $field->getName() . "', " .$field->getPHP(true)

            Otherwise looks good

            Show
            andyjdavis Andrew Davis added a comment - [Y] Syntax [Y] Output [?] Whitespace [Y] Language [NA] Databases [?] Testing [NA] Security [NA] Documentation [Y] Git [Y] Sanity check From the testing instructions. "Make sure it works and the coding guidelines are respected" How would one go about doing that? I feel like you're missing a step here. You didn't introduce it but there is a space missing on line 522 of admin/tool/xmldb/actions/view_table_php/view_table_php.class.php $field->getName() . "', " .$field->getPHP(true) Otherwise looks good
            Hide
            fred Frédéric Massart added a comment -

            Thanks Andrew, I've fixed the missing white space and edited my testing instructions. Cheers!

            Show
            fred Frédéric Massart added a comment - Thanks Andrew, I've fixed the missing white space and edited my testing instructions. Cheers!
            Hide
            damyon Damyon Wiese added a comment -

            The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

            Cheers!

            Show
            damyon Damyon Wiese added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. Cheers!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (master only), thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (master only), thanks!
            Hide
            dmonllao David Monllaó added a comment -

            It passes, the only no-PHP code I've found is a DON'T USE THIS FUNCTION (IT'S ONLY EXPERIMENTAL). SOME DBs DON'T SUPPORT IT! in table->rename_index and table->rename_key

            Show
            dmonllao David Monllaó added a comment - It passes, the only no-PHP code I've found is a DON'T USE THIS FUNCTION (IT'S ONLY EXPERIMENTAL). SOME DBs DON'T SUPPORT IT! in table->rename_index and table->rename_key
            Hide
            damyon Damyon Wiese added a comment -

            Congratulations this fix has been added to Moodle!

            You may want to dedicate this issue to someone special on this Valentines day.

            Thanks!

            Show
            damyon Damyon Wiese added a comment - Congratulations this fix has been added to Moodle! You may want to dedicate this issue to someone special on this Valentines day. Thanks!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/13