Uploaded image for project: 'Kernel'
  1. Kernel
  2. KNL-213

BaseResourceEdit return non url-encoded urls

    Details

    • Type: Bug Bug
    • Status: 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

        Gliffy Diagrams

          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:

                  Development