Moodle
  1. Moodle
  2. MDL-29378

survey-COLLES (Preferred) and survey-COLLES (Actual) are not different from survey-COLLES (Preferred and Actual)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1, 2.2
    • Fix Version/s: 2.4.7, 2.5.3
    • Component/s: Survey
    • Labels:
      None
    • Rank:
      18902

      Description

      I do not remember how it should be but for sure they can not be equals.
      Probably, they used to have a lot of "I prefer that" and "I find that" .... or something like that.
      It may be matter of checking old lang files only.

        Issue Links

          Activity

          Hide
          Martin Dougiamas added a comment -

          Yes there is a nasty regression here. I can't believe it's not been noticed more before now!!

          It should look like this http://surveylearning.moodle.com/mod/survey/view.php?id=3

          Show
          Martin Dougiamas added a comment - Yes there is a nasty regression here. I can't believe it's not been noticed more before now!! It should look like this http://surveylearning.moodle.com/mod/survey/view.php?id=3
          Hide
          Joseph Rézeau added a comment -

          Martin "I can't believe it's not been noticed more before now!"
          Maybe this has not been noticed because the Survey activity is hardly ever used by moodlers, given its non-customizable nature.

          Show
          Joseph Rézeau added a comment - Martin "I can't believe it's not been noticed more before now!" Maybe this has not been noticed because the Survey activity is hardly ever used by moodlers, given its non-customizable nature.
          Hide
          Martin Dougiamas added a comment -

          Thanks Joseph, helpful.

          Show
          Martin Dougiamas added a comment - Thanks Joseph, helpful.
          Hide
          David Mudrak added a comment -

          given its non-customizable nature

          Well. The other view might be that COLLES is a standardized survey tool, with its own beackground in research, methodology, data processing and everything. Many people think that any questionnaire they quickly sum up would produce good data. Often, having a standard as a reference has real benefits.

          Show
          David Mudrak added a comment - given its non-customizable nature Well. The other view might be that COLLES is a standardized survey tool, with its own beackground in research, methodology, data processing and everything. Many people think that any questionnaire they quickly sum up would produce good data. Often, having a standard as a reference has real benefits.
          Hide
          Joseph Rézeau added a comment -

          @Martin and David,
          I was not questioning the validity of the Moodle COLLES survey activity, only pointing out that, from what I gather from the moodle.org Survey forum, it seems to be rarely used as is, and most posters would like to be able to customize it.
          Which is in fact what is going to happen in the forthcoming Survey2 module, which will combine the standard COLLES survey with fully customizable surveys.
          Joseph

          Show
          Joseph Rézeau added a comment - @Martin and David, I was not questioning the validity of the Moodle COLLES survey activity, only pointing out that, from what I gather from the moodle.org Survey forum, it seems to be rarely used as is, and most posters would like to be able to customize it. Which is in fact what is going to happen in the forthcoming Survey2 module, which will combine the standard COLLES survey with fully customizable surveys. Joseph
          Hide
          David Mudrak added a comment -

          This is a regression caused by 5196df589b0fbcead4a0943c8e7b227f8a98c897

          Show
          David Mudrak added a comment - This is a regression caused by 5196df589b0fbcead4a0943c8e7b227f8a98c897
          Show
          Martin Dougiamas added a comment - ie https://github.com/moodle/moodle/commit/5196df589b0fbcead4a0943c8e7b227f8a98c897
          Hide
          David Mudrak added a comment -

          So, I have somewhat progress here. It's really hard to deduct what Andrew was describing in the commit message of the regressive change. But when his patch is reverted, both "I prefer that" and "I found that" lines appear again. It's apparent that the patch is wrong as the subquestion's type does not determine if there should be one or two answer lines. It's always the parent (master) question (such as "collesm1") whose type determines this. But I would really like to understand Andrew's reasons for the change.

          As usually, other issues raise as one starts to play with it. Apparently, since MDL-7501, rows are not highlighted any more (see the file reverted.png). I'm not really sure that the suggested fix in MDL-7501 was good. If nothing, than the multi COLLES now has at least 48 items and not 24 as the description says. However, I tried to fix the row classes calculation too. The result is in reverted+highlighted.png file.

          Show
          David Mudrak added a comment - So, I have somewhat progress here. It's really hard to deduct what Andrew was describing in the commit message of the regressive change. But when his patch is reverted, both "I prefer that" and "I found that" lines appear again. It's apparent that the patch is wrong as the subquestion's type does not determine if there should be one or two answer lines. It's always the parent (master) question (such as "collesm1") whose type determines this. But I would really like to understand Andrew's reasons for the change. As usually, other issues raise as one starts to play with it. Apparently, since MDL-7501 , rows are not highlighted any more (see the file reverted.png). I'm not really sure that the suggested fix in MDL-7501 was good. If nothing, than the multi COLLES now has at least 48 items and not 24 as the description says. However, I tried to fix the row classes calculation too. The result is in reverted+highlighted.png file.
          Hide
          Martin Dougiamas added a comment - - edited

          I think Reverted+highlighted looks OK. It's certainly much better and usable enough. And probably accessible too.

          So assuming that the results all get saved and results are stored/shown correctly then my +1 for it!

          Show
          Martin Dougiamas added a comment - - edited I think Reverted+highlighted looks OK. It's certainly much better and usable enough. And probably accessible too. So assuming that the results all get saved and results are stored/shown correctly then my +1 for it!
          Hide
          David Mudrak added a comment -

          Submitting form integration. See commit messages for more explanation.

          Show
          David Mudrak added a comment - Submitting form integration. See commit messages for more explanation.
          Hide
          Sam Hemelryk added a comment -

          Thanks David this has been integrated now.

          Show
          Sam Hemelryk added a comment - Thanks David this has been integrated now.
          Hide
          Joseph Rézeau added a comment -

          Tested the fix in Moodle 2.5, looks OK to me.

          Show
          Joseph Rézeau added a comment - Tested the fix in Moodle 2.5, looks OK to me.
          Hide
          David Mudrak added a comment -

          Argh! Sam Hemelryk, I just realized there was a trivial mistake in the integrated branch yet. I added yet another commit on top of each branch. Can you please re-emerge them or alternatively cherry-pick the last commit with the message "MDL-29378 Fix the recent string modification". I'm sorry.

          Show
          David Mudrak added a comment - Argh! Sam Hemelryk , I just realized there was a trivial mistake in the integrated branch yet. I added yet another commit on top of each branch. Can you please re-emerge them or alternatively cherry-pick the last commit with the message " MDL-29378 Fix the recent string modification". I'm sorry.
          Hide
          Sam Hemelryk added a comment -

          No probs David, I've integrated the updated branches now

          Show
          Sam Hemelryk added a comment - No probs David, I've integrated the updated branches now
          Hide
          Marina Glancy added a comment -

          wow, this took a long time to test. All surveys are working fine.

          I noticed that the whole UI of surveys is so unfriendly, but I'm not going to create any issues because it's not likely that it is widely used anyway

          Show
          Marina Glancy added a comment - wow, this took a long time to test. All surveys are working fine. I noticed that the whole UI of surveys is so unfriendly, but I'm not going to create any issues because it's not likely that it is widely used anyway
          Hide
          Damyon Wiese added a comment -

          This issue along with 77 of it's friends has been sent upstream and released to the world.

          Thankyou for your contribution.

          Show
          Damyon Wiese added a comment - This issue along with 77 of it's friends has been sent upstream and released to the world. Thankyou for your contribution.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: