Uploaded image for project: 'Moodle Community Sites'
  1. Moodle Community Sites
  2. MDLSITE-2132

RFC: Clarify the line wrapping/indentation rules

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Low Low
    • Coding style
    • None

      This issue has been hijacked to gather together another of indentation style related issues which have been made subtasks of this general issue.
      —
      Right now, the rules about wrapping lines in code says clearly that a 8-spaces indentation must be used, but later the example shows the use for a control structure (if), not for general use were people could think that 4-spaces is the norm.

      http://docs.moodle.org/dev/Coding_style#Wrapping_lines

      This is a request to clarify that by:

      1) Explicitly saying that the 8-spaces rule is for any code (general and control structures).
      2) Providing one more example of the rule in general code.

      Ciao

      PS: All this comes from CONTRIB-3562 and current inability for the codechecker to verify that indentation. So for now, better make the rule clearer.

        1.
        		 Improve control structure wrapping style Sub-task Open Dan Poltawski
        2.
        		 Clarify Coding style Sub-task Open Dan Poltawski
        3.
        		 RFC: Coding style for wrapping the code Sub-task Open Dan Poltawski
        4.
        		 How to indent line with many object operators Sub-task Closed Eloy Lafuente (stronk7)

            mudrd8mz David Mudrák (@mudrd8mz)
            stronk7 Eloy Lafuente (stronk7)
            Votes:
            2 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.