Uploaded image for project: 'Plugins'
  1. Plugins
  2. CONTRIB-5240

Fix CamelCase problem in local_codechecker

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.7
    • Fix Version/s: 2.7.2
    • Component/s: Local: Code checker
    • Labels:
      None
    • Affected Branches:
      MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE

      Description

      It seems that CamelCase problems in codechecker continue hitting different system (case-sensitive) here and there. That's specially noticeable since recent inclusion of a new "moodle" standard sniff using a "PHPCompatibility" standard sniff, that is causing problems both under linux (CI servers) and windows (human users of the local_codechecker).

      See: https://github.com/moodlehq/moodle-local_codechecker/pull/17

      So this is about to get rid of such case-sensitiveness problems forever. And the plan is:

      1) Rename the "phpcompatibility" to match the official "PHPCompatibility" name of the standard and its classes.
      2) Rename the "Moodle" to match the official "moodle" name of the standard and its classes.

      That way, there won't be any problem/confusion anymore. That's all.

      Note: While this is being fixed, all the related CiBoT jobs in the CI servers have been halted to avoid getting wrong results for ongoing issues.

        Attachments

          Activity

            People

            Assignee:
            stronk7 Eloy Lafuente (stronk7)
            Reporter:
            stronk7 Eloy Lafuente (stronk7)
            Tester:
            Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
            Participants:
            Component watchers:
            moodle.com, Eloy Lafuente (stronk7)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              8/Sep/14