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

missing double quotes in /grade/lib.php

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8.2
    • Fix Version/s: 1.8.2
    • Component/s: Gradebook
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE

      Description

      line 2249 currently reads:
      $row .= '<td align=right>'.$grades_by_student[$student]['student_data']['weighted'].'%</td>';
      it is missing the double quotes around right as in the other instances (see line 2246 for an example)

      line 2249 should read:
      $row .= '<td align="right">'.$grades_by_student[$student]['student_data']['weighted'].'%</td>';

        Gliffy Diagrams

          Activity

          Hide
          aborrow Anthony Borrow added a comment -

          Jeff - As I continue working on merging the GBPv2 code with the latest updates I caught this typo. Peace - Anthony

          Show
          aborrow Anthony Borrow added a comment - Jeff - As I continue working on merging the GBPv2 code with the latest updates I caught this typo. Peace - Anthony
          Hide
          aborrow Anthony Borrow added a comment -

          diff file to apply suggested change

          Show
          aborrow Anthony Borrow added a comment - diff file to apply suggested change
          Hide
          aborrow Anthony Borrow added a comment -

          MartinD - I would be happy to correct this types of issues; however, I will need to be able to commit them. I have a much better handle on CVS and the various branches especially now that I am using PHP Eclipse. Peace - Anthony

          Show
          aborrow Anthony Borrow added a comment - MartinD - I would be happy to correct this types of issues; however, I will need to be able to commit them. I have a much better handle on CVS and the various branches especially now that I am using PHP Eclipse. Peace - Anthony
          Hide
          aborrow Anthony Borrow added a comment -

          fixed in 18STABLE

          Show
          aborrow Anthony Borrow added a comment - fixed in 18STABLE
          Hide
          aborrow Anthony Borrow added a comment -

          fixed in 18STABLE - not an issue in HEAD

          Show
          aborrow Anthony Borrow added a comment - fixed in 18STABLE - not an issue in HEAD

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                8/Jul/07