Kernel
  1. Kernel
  2. KNL-213

BaseResourceEdit return non url-encoded urls

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.18
    • Fix Version/s: 1.1.8
    • Component/s: Impl
    • Labels:
      None
    • Previous Issue Keys:

      Description

      BaseResourceEdit (and other classes in BaseContentService.java) returns urls that are not url encoded, thus breaking on filenames with characters that have to be encoded.

      For us this breaks uploaded files in atleast syllabus and test&quizzes

        Issue Links

          Activity

          Hide
          Matthew Buckett added a comment -
          A merge to 1.0.x is need to fix what issue in 2.6?
          As Stephen mentioned I think this is a risky issue to merge back into 2.6/K1 due to the risk of double escaping.
          Show
          Matthew Buckett added a comment - A merge to 1.0.x is need to fix what issue in 2.6? As Stephen mentioned I think this is a risky issue to merge back into 2.6/K1 due to the risk of double escaping.
          Hide
          Anthony Whyte added a comment -
          Bumped to 1.0.14. However, a more formal decision should be rendered regarding whether or not to include this fix in 1.0.x given the range of 2.6 tools that require fixing in order to implement the fix properly.
          Show
          Anthony Whyte added a comment - Bumped to 1.0.14. However, a more formal decision should be rendered regarding whether or not to include this fix in 1.0.x given the range of 2.6 tools that require fixing in order to implement the fix properly.
          Hide
          David Horwitz added a comment -
          due to the impact of this change it won't be fixed at this point in the 1.0 series
          Show
          David Horwitz added a comment - due to the impact of this change it won't be fixed at this point in the 1.0 series
          Hide
          Nicola Monat-Jacobs added a comment -
          Since this was never back-ported to the 1.0.x branch, can we remove the '1.0.14' fix version?
          Show
          Nicola Monat-Jacobs added a comment - Since this was never back-ported to the 1.0.x branch, can we remove the '1.0.14' fix version?
          Hide
          Stephen Marquard added a comment -
          Changed Fix Version as per discussion above.
          Show
          Stephen Marquard added a comment - Changed Fix Version as per discussion above.

            People

            • Assignee:
              Jim Eng
              Reporter:
              Klas Lindfors
            • Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: