Moodle
  1. Moodle
  2. MDL-32597

typo in local/readme.txt for version.php location

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Libraries
    • Labels:

      Description

      reported by someone in IRC:

      • /local/pluginname/db/version.php - version of script (must be incremented after changes)
        should be:
      • /local/pluginname/version.php - version of script (must be incremented after changes)

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Petr Skoda added a comment -

            oh, thanks!!

            Show
            Petr Skoda added a comment - oh, thanks!!
            Hide
            Eloy Lafuente (stronk7) 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.

            TIA and ciao

            Show
            Eloy Lafuente (stronk7) 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. TIA and ciao
            Hide
            Sam Hemelryk added a comment -

            Thanks guys, this has been integrated now

            Show
            Sam Hemelryk added a comment - Thanks guys, this has been integrated now
            Hide
            Andrew Davis added a comment -

            "do I really need to put something in here?"

            Yes. Yes you do.

            The point of testing is to not only check that the fix works as it should but that the integrators integrated it correctly into all branches.

            The change exists in master, 2.2 stable and 2.1 stable. Passing.

            Show
            Andrew Davis added a comment - "do I really need to put something in here?" Yes. Yes you do. The point of testing is to not only check that the fix works as it should but that the integrators integrated it correctly into all branches. The change exists in master, 2.2 stable and 2.1 stable. Passing.
            Hide
            Eloy Lafuente (stronk7) added a comment -

            UPDATE tracker_issues
               SET status = 'Closed',
                  comment = 'Thanks!'
            WHEN participants = 'Did a gorgeous work'
            

            This landed upstream some hours ago (some - me - developer fell slept in the sofa yesterday before spamming this).

            Show
            Eloy Lafuente (stronk7) added a comment - UPDATE tracker_issues SET status = 'Closed', comment = 'Thanks!' WHEN participants = 'Did a gorgeous work' This landed upstream some hours ago (some - me - developer fell slept in the sofa yesterday before spamming this).

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: