Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.3, 2.1
    • Fix Version/s: 2.0.4
    • Component/s: Feedback
    • Labels:
      None
    • Environment:
      php compiled without php5-bcmath
    • Testing Instructions:
      Hide

      create two questions in any feedback. One multichoice and one multichoicerated.
      do some submitts.
      Go to the analysis-page.
      If bcmod() is available so the printed bars are in different colours.
      If bcmod() isn't available so the printed bars are all the same colour.

      Show
      create two questions in any feedback. One multichoice and one multichoicerated. do some submitts. Go to the analysis-page. If bcmod() is available so the printed bars are in different colours. If bcmod() isn't available so the printed bars are all the same colour.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-28014_head
    • Rank:
      17696

      Description

      There are two calls of bcmod() in item/multichoice/lib.php and item/multichoicerated/lib.php but this function isn't available on each installation.
      Better is to use the % for modulus then bcmod().
      This was just used in a very early time I started php programming .

        Activity

        Hide
        Sam Hemelryk added a comment -

        Hi Andreas,

        This gets my +1 for integration however we are within the code freeze before 2.1 release now.
        I will talk to Eloy later today if he is happy for it to go it I'm happy to integrate it - otherwise it will have to wait till after the release of 2.1.

        Cheers
        Sam

        Show
        Sam Hemelryk added a comment - Hi Andreas, This gets my +1 for integration however we are within the code freeze before 2.1 release now. I will talk to Eloy later today if he is happy for it to go it I'm happy to integrate it - otherwise it will have to wait till after the release of 2.1. Cheers Sam
        Hide
        Eloy Lafuente (stronk7) added a comment -

        My +1 to this, seems "innocent" enough to have in integrated.

        Thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - My +1 to this, seems "innocent" enough to have in integrated. Thanks!
        Hide
        Sam Hemelryk added a comment -

        Integration achieved

        Show
        Sam Hemelryk added a comment - Integration achieved
        Hide
        Rajesh Taneja added a comment -

        Works Great
        Thanks for providing the fix Andreas.

        Show
        Rajesh Taneja added a comment - Works Great Thanks for providing the fix Andreas.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Yay, this is now part of the just released Moodle 2.1 ! Thanks for all the hard work!

        Show
        Eloy Lafuente (stronk7) added a comment - Yay, this is now part of the just released Moodle 2.1 ! Thanks for all the hard work!

          People

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

            Dates

            • Created:
              Updated:
              Resolved: