Uploaded image for project: 'CMS'
  1. CMS
  2. CMS-7206

SendMailFunction does not handle correctly "cms.url" config parameter when it ends with /index.html

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 4.2.0
    • 4.0.0
    • None
    • None
    • 4.2 M7

        [CMS-7206] SendMailFunction does not handle correctly "cms.url" config parameter when it ends with /index.html

        The new regexp does not work on FF

        new RegExp ("https?://.*[^/](?<!index.html)$")
        SyntaxError: invalid regexp group

         

        Laurence Aumeunier added a comment - The new regexp does not work on FF new RegExp ("https?://.*[^/](?<!index.html)$") SyntaxError: invalid regexp group  

        I think it is more appropriate to simply have a regexp validator to ensuire this parameter always have the same format (it could even test for 'index.html' to refuse)

        Raphaël Franchet added a comment - I think it is more appropriate to simply have a regexp validator to ensuire this parameter always have the same format (it could even test for 'index.html' to refuse)

        I proposed a content and received a mail with the following url:

        .../cms/index.html/www/index.html?uitool=uitool-page,id:%27page://666b194b-520a-4358-957e-78fb644a06cc%27.

        Edit:
        A lot of cases where "cms.url" ends with /index.hml seems to be handled, but not in the workflow SendMailFunction in v4.
        In the SendMailFunction of 3.8 the cms.url config parameter is not used so it might due to a merge.

        Thibaut Rizzi (Inactive) added a comment - I proposed a content and received a mail with the following url: .../cms/index.html/www/index.html?uitool=uitool-page,id:%27page://666b194b-520a-4358-957e-78fb644a06cc%27. Edit : A lot of cases where "cms.url" ends with /index.hml seems to be handled, but not in the workflow SendMailFunction in v4. In the SendMailFunction of 3.8 the cms.url config parameter is not used so it might due to a merge.

          raphael Raphaël Franchet
          trizzi Thibaut Rizzi (Inactive)
          Votes:
          0 Vote for this issue
          Watchers:
          3 Start watching this issue

            Created:
            Updated:
            Resolved: