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

Multisite : redirection sur le /

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 2.1.10
    • None
    • Workspace : CMS
    • None
    • Operating System: Windows XP
      Platform: PC

      actuellement le / est redirigé vers /fr/contributor
      il faudrait que /

      {nomdusite} ou / en non-multisite doit rediriger vers /{nomdusite}

      /

      {languepardefaut}

      /contributor

          [CMS-336] Multisite : redirection sur le /

          ajout de UnknownSiteException à la configuration
          ajout d'une vérification de l'existance du site dans le decoding matcher

          Raphaël Franchet added a comment - ajout de UnknownSiteException à la configuration ajout d'une vérification de l'existance du site dans le decoding matcher

          la bonne solution est sans doute de créer une SiteDoesNotExistException qui
          sera traitée en tant que telle.
          Il faut faire ca en 2.1 pour pas avoir à peter l'API apres

          Cédric Damioli added a comment - la bonne solution est sans doute de créer une SiteDoesNotExistException qui sera traitée en tant que telle. Il faut faire ca en 2.1 pour pas avoir à peter l'API apres

          La NPE est corrigée, elle est remplacée par une illegalargumentexception.
          Du coup, comme on prend la skin, le site configuration gueule que le site
          existe pas.
          ça résoud un peu le bug, mais par effet de bord, ce qui est pas bien !

          Raphaël Franchet added a comment - La NPE est corrigée, elle est remplacée par une illegalargumentexception. Du coup, comme on prend la skin, le site configuration gueule que le site existe pas. ça résoud un peu le bug, mais par effet de bord, ce qui est pas bien !

          si on met le nom d'un site qui n'existe pas, on se ramasse une NPE du site
          configuration.
          faudrait pitet travailler en amont !

          Raphaël Franchet added a comment - si on met le nom d'un site qui n'existe pas, on se ramasse une NPE du site configuration. faudrait pitet travailler en amont !

          Les redirections des sites marchent pas quand on met un / derriere, c pas
          bien !

          ex:

          /cms/default fonctionne
          /cms/default/ ne fonctionne pas

          Raphaël Franchet added a comment - Les redirections des sites marchent pas quand on met un / derriere, c pas bien ! ex: /cms/default fonctionne /cms/default/ ne fonctionne pas

          Le / de la racine ne doit pas rediriger vers /fr/contributor en multisites

          Raphaël Franchet added a comment - Le / de la racine ne doit pas rediriger vers /fr/contributor en multisites

            cedric Cédric Damioli
            raphael Raphaël Franchet
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: