Labels Milestones

Refactoring de la fonction _T_ou_typo() #14

Merged

maieul merged 6 commits from refactor_t_ou_typo into master 2 months ago

Conversation 2 Commits 6 Files Changed 2

cerdic commented 2 months ago

Owner

Et jeux de tests unitaire pour s'assurer qu'on ne casse rien et constituer un début de spec

cf

• spip-contrib-extensions/saisies#264
• spip-contrib-extensions/saisies#259
• spip-contrib-extensions/formidable#140

Et jeux de tests unitaire pour s'assurer qu'on ne casse rien et constituer un début de spec cf * https://git.spip.net/spip-contrib-extensions/saisies/issues/264 * https://git.spip.net/spip-contrib-extensions/saisies/issues/259 * https://git.spip.net/spip-contrib-extensions/formidable/issues/140

cerdic added 6 commits 2 months ago

729254f842 fix: ajouter 2 arguments connect et env a la fonction _T_ou_typo() qui sont passés directement à l'appel de typo() pour permettre les tests unitaires dans toutes les situations

6df0d65ddf tests: un jeu de tests unitaires pour _T_ou_Typo(), qui reflete le fonctionnement actuel sauf pour le cas _ecrire ou la fonction déclenche de l'echappement suspect alors qu'on en veut pas

3a23f0d192 tests: completer avec des entrees de type string contenant des raccourcis SPIP

7c0ab1d9ab refactor: simplifier les tests dans _T_ou_typo

7af57f6072 fix: passer un connect non null a typo() pour ne pas déclencher une sursecurisation

9cc8be6794 fix: forcer espace_prive=false dans le env pour ne pas declencher securiser_html_suspect() sans arrêt, car les entrées sont en principe non fournies par les utilsateurs + passer interdire_scripts() sur le résultat dans tous les cas ...

A noter : en cas de `$GLOBALS['filtrer_javascript'] == -1` le résultat passer quand même dans safehtml

cerdic referenced this pull request from spip-contrib-extensions/saisies 2 months ago

_T_ou_typo provoque des échappements indus dans l'espace privé en SPIP 4.2 #264

maieul commented 2 months ago

Collaborator

Merci @cerdic.

J'ai

1. Testé en réel
2. Regardé le diff
3. Regardé les tests unitaires

Pour moi tout est oki ! On peut merger :) Et quand ce sera validé je reporterai dans saisies (rah, rah cette duplication de code)...

Merci @cerdic. J'ai 1. Testé en réel 2. Regardé le diff 3. Regardé les tests unitaires Pour moi tout est oki ! On peut merger :) Et quand ce sera validé je reporterai dans saisies (rah, rah cette duplication de code)...

maieul approved these changes 2 months ago

cerdic commented 2 months ago

Poster

Owner

cool

cool

maieul merged commit 9cc8be6794 into master 2 months ago

maieul deleted branch refactor_t_ou_typo 2 months ago

maieul referenced this issue from a commit 2 months ago

fix: refactor de `_T_ou_typo()` pour rendre plus robuste.

Reviewers

maieul approved these changes 2 months ago

The pull request has been merged as 9cc8be6794.

You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.

git checkout -b refactor_t_ou_typo master

git pull origin refactor_t_ou_typo

Step 2:

Merge the changes and update on Gitea.

git checkout master

git merge --no-ff refactor_t_ou_typo

git push origin master

Sign in to join this conversation.

Reviewers

Request review

No reviewers

maieul

Labels

Apply labels

Clear labels   

bug

Something is not working   

duplicate

This issue or pull request already exists   

enhancement

New feature   

help wanted

Need some help   

invalid

Something is wrong   

question

More information is needed   

wontfix

This won't be fixed

No Label

bug

duplicate

enhancement

help wanted

invalid

question

wontfix

Milestone

Set milestone

Clear milestone

No items

No Milestone

Assignees

Assign users

Clear assignees

No Assignees

2 Participants

Notifications

Subscribe

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: spip-contrib-extensions/spip-bonux#14

Write Preview

Loading…

Cancel

Save

There is no content yet.