Moodle
  1. Moodle
  2. MDL-25299

MNET: Language string not displaying correctly

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1.4, 2.2.1, 2.3
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: MNet
    • Labels:
    • Testing Instructions:
      Hide
      1. Enable MNet on two sites
      2. On site one proceed to start adding the second as a peer
      3. When reviewing the details before saving delete the key and attempt to save
      4. Check you get a required notice
      5. Refresh the browser so you get the key back and save
      6. Now edit the saved peer and remove the public key
      7. Ensure you get an error again
      Show
      Enable MNet on two sites On site one proceed to start adding the second as a peer When reviewing the details before saving delete the key and attempt to save Check you get a required notice Refresh the browser so you get the key back and save Now edit the saved peer and remove the public key Ensure you get an error again
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      wip-MDL-25299-m23
    • Rank:
      7138

      Description

      Hi Sam,

      Not sure who is responsible for the mnet component so assigned to you

      When you submit the form when editing a host with an empty public key you are displayed with an incomplete language string.

      "The key is not a valid SSL key. (3: The subject of the certificate:
      {$a->subject}
      does not match the host it came from:
      {$a->host}.
      )"

      Regards,

      Mark

        Activity

        Hide
        Sam Hemelryk added a comment -

        Putting this up for peer-review now. I've implemented better validation of the public key field.

        Show
        Sam Hemelryk added a comment - Putting this up for peer-review now. I've implemented better validation of the public key field.
        Hide
        Ankit Agarwal added a comment -

        Hi Sam,
        Master compare url is giving 404 errors
        Had a look on stables and changes look good to me.
        Thanks

        Show
        Ankit Agarwal added a comment - Hi Sam, Master compare url is giving 404 errors Had a look on stables and changes look good to me. Thanks
        Hide
        Sam Hemelryk added a comment -

        Thanks Ankit, fixed up the missing branch and this is up for integration now.

        Cheers
        Sam

        Show
        Sam Hemelryk added a comment - Thanks Ankit, fixed up the missing branch and this is up for integration now. Cheers Sam
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Integrated, thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
        Hide
        Andrew Davis added a comment -

        Works as described although I had to go back to the main peers page and start the add peer process over again to get the key back. Passing.

        There are some fairly obvious usability problems, no cancel button on the edit peer page, "Peers" in the breadcrumbs isnt clickable, but I guess that isnt worth bothering with what with all of this eventually being replaced by web services.

        Show
        Andrew Davis added a comment - Works as described although I had to go back to the main peers page and start the add peer process over again to get the key back. Passing. There are some fairly obvious usability problems, no cancel button on the edit peer page, "Peers" in the breadcrumbs isnt clickable, but I guess that isnt worth bothering with what with all of this eventually being replaced by web services.
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Some changes to Moodle should be milestones in the project by themselves.

        This is not the case and your fix is not so important, but your collaboration is highly appreciated, thanks!

        Closing as fixed, ciao

        Show
        Eloy Lafuente (stronk7) added a comment - Some changes to Moodle should be milestones in the project by themselves. This is not the case and your fix is not so important, but your collaboration is highly appreciated, thanks! Closing as fixed, ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: