Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.5, 2.2.2, 2.3
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Feedback
    • Labels:
    • Testing Instructions:
      Hide
      1. you need a course with one or more studends
      2. create a non-anonymous feedback instance and one or more questions
      3. create one or more questions
      4. login as student and submit a feedback
      5. login as teacher and go "Show responses". You will see a table with the initialbars for first- and surname
      6. select a letter from the initialbars so that no user will be find
      7. the initialbar still should be there.
      Show
      you need a course with one or more studends create a non-anonymous feedback instance and one or more questions create one or more questions login as student and submit a feedback login as teacher and go "Show responses". You will see a table with the initialbars for first- and surname select a letter from the initialbars so that no user will be find the initialbar still should be there.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32624_master_wip
    • Rank:
      39552

      Description

      In show_entries.php the initialbar disappears if a letter is select on what no user is found.

        Activity

        Hide
        Eloy Lafuente (stronk7) added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Sam Hemelryk added a comment -

        Thanks Andreas this has been integrated now

        Show
        Sam Hemelryk added a comment - Thanks Andreas this has been integrated now
        Hide
        Michael de Raadt added a comment -

        Test result: Success

        Tested in 2.1, 2.2 and master.

        Show
        Michael de Raadt added a comment - Test result: Success Tested in 2.1, 2.2 and master.
        Hide
        Eloy Lafuente (stronk7) added a comment -
        UPDATE tracker_issues
           SET status = 'Closed',
              comment = 'Thanks!'
        WHEN participants = 'Did a gorgeous work'
        

        This landed upstream some hours ago (some - me - developer fell slept in the sofa yesterday before spamming this).

        Show
        Eloy Lafuente (stronk7) added a comment - UPDATE tracker_issues SET status = 'Closed', comment = 'Thanks!' WHEN participants = 'Did a gorgeous work' This landed upstream some hours ago (some - me - developer fell slept in the sofa yesterday before spamming this).

          People

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

            Dates

            • Created:
              Updated:
              Resolved: