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

Clean up Feedback modules coding style

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • 2.2
    • 2.2
    • Feedback
    • MOODLE_22_STABLE
    • MOODLE_22_STABLE
    • MDL-29804_master
    • Hide

      These changes consist primarily of whitespace fixes and as such only a very little bit of code has changed.

      1. Test that you can still add/edit/delete feedback modules as an admin or teacher
      2. Test that as a student you can submit feedback
      3. Test the reporting as an admin/teacher

      Testing the issue with new function xxx_param_array()

      1. turn debug on for developer
      2. create a feedback and an item multichoice "Multiple choice - multiple answers" (checkbox) with two or more options
      3. go to complete this feedback and check one or more checkboxes
      4. while submitting no error should be raised
        The special changes are done in feedback/lib.php in functions feedback_check_values, feedback_create_values, feedback_update_values
        and in feedback/item/multichoice/lib.php in function item_array_to_string.
      Show
      These changes consist primarily of whitespace fixes and as such only a very little bit of code has changed. Test that you can still add/edit/delete feedback modules as an admin or teacher Test that as a student you can submit feedback Test the reporting as an admin/teacher Testing the issue with new function xxx_param_array() turn debug on for developer create a feedback and an item multichoice "Multiple choice - multiple answers" (checkbox) with two or more options go to complete this feedback and check one or more checkboxes while submitting no error should be raised The special changes are done in feedback/lib.php in functions feedback_check_values, feedback_create_values, feedback_update_values and in feedback/item/multichoice/lib.php in function item_array_to_string.

      The feedback modules coding style needs to be cleaned up in the master branch.
      Things such as spacing, variable names and phpdocs can be improved.

      The coding style can be found in the docs: http://docs.moodle.org/dev/Coding_style
      Also worth looking at is Tim's code checker: https://github.com/timhunt/moodle-local_codechecker

            grabs Andreas Grabs
            samhemelryk Sam Hemelryk
            Sam Hemelryk Sam Hemelryk
            Ankit Agarwal Ankit Agarwal
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved:

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