search

nursit / bank

module de paiement bancaire multi prestataires & stockage des transactions pour SPIP
20 stars 22 forks source link

.htaccess obligatoire pour paypal express ? #24

Closed phenix-factory closed 4 years ago

phenix-factory commented 8 years ago

Hello,

On a remarqué un comportement étrange : si le htacces n'est pas activé, le payement par paypal express échoue car bank utilise les URL .api qui sont dedans.

Du coup il faudrait prévenir l'utilisateur si son htaccess n'est pas activé.

Cerdic commented 8 years ago

normalement le plugin regarde si le fichier .htaccess est là et si il y a bien la RewriteRule sur les api avant d'utiliser ce type d'URL. Mais c'est vrai que si le fichier n'est pas pris en compte par apache ça bloque. Il faudrait un test plus robuste

phenix-factory commented 8 years ago

Peut être qu'il faudrai commencer par avoir une fonction qui test que le htaccess de SPIP fonctionne dans le core de SPIP. Ainsi, tout les plugins pourraient en profiter.

Peut être aussi qu'on pourrai personnaliser l'erreur des pages API pour qu'elles avertissent que le htaccess n'est peut être pas activé ?

J'ouvre un ticket pour le core ?

Cerdic commented 4 years ago

J'ai amélioré la detection de la présence de la rewrite rule, pour prendre en compte si elle est commentée ou non, mais il reste que le htaccess doit etre fonctionnel si il est présent, en effet (sinon c'est un cas un peu borderline, qu'on ne traitera pas ici, et qui de toute façon fera coincer toutes les méthodes de paiement)