spip
/
spip
23
12
Fork 16
Code Issues Pull Requests 11 Releases 137 Wiki Activity
Labels Milestones
New Pull Request

#128 #4445 : savoir si espace privé en JS + début d'utilisation d'un objet spipConfig

Open
rastapopoulos wants to merge 2 commits from dev/issue_4445_test_prive_js into master
Conversation 0 Commits 2 Files Changed 3
rastapopoulos commented 1 week ago
Owner

Cette PR sert à résoudre #4445, et dans le même temps commence à résoudre #4531 mais pas complètement.

Une fonction (cloisonnée donc) jQuery.spip.test_espace_prive() permet de savoir tout comme en PHP.

Derrière ça commence à utiliser un objet spipConfig indépendant de jQuery, pour y recueillir toutes sortes d'infos sans en être dépendant.

Au passage on permet aussi de lancer les API depuis ecrire/ (il faudra être à jour de son htaccess, tout comme quand on a introduit les API), pour doublement résoudre ce point précis (mais ce n'était qu'un cas possible qui nécessitait ce ticket, qui reste avant tout sur le JS).

Pour #4445 ça suffit, ça résout et on pourra fermer.

Pour vraiment résoudre #4531 il faudrait :

  • que cette objet spipConfig soit rempli dans une fonction PHP d'abord, possiblement passant dans un nouveau pipeline dédié afin que des plugins puissent compléter en PHP des infos utiles à accéder en JS ensuite (config, traduction, etc)
  • qu'il soit inséré alors en JS dans cette fonction PHP aussi, appelé au tout début de insert_head et de header_prive, alors que là j'ai commencé à le remplir dans le squelette, tout simplement au même endroit que là où sont définis pour l'instant les autres variables JS de l'admin
  • en conséquence ces autres variables JS (à garder pour rien péter) et leur doublonnage dans spipConfig, devraient donc aussi être dans cette fonction PHP et non plus là dans ce squelette
Cette PR sert à résoudre #4445, et dans le même temps commence à résoudre #4531 mais pas complètement. Une fonction (cloisonnée donc) jQuery.spip.test_espace_prive() permet de savoir tout comme en PHP. Derrière ça commence à utiliser un objet spipConfig indépendant de jQuery, pour y recueillir toutes sortes d'infos sans en être dépendant. Au passage on permet aussi de lancer les API depuis ecrire/ (il faudra être à jour de son htaccess, tout comme quand on a introduit les API), pour doublement résoudre ce point précis (mais ce n'était qu'un cas possible qui nécessitait ce ticket, qui reste avant tout sur le JS). Pour #4445 ça suffit, ça résout et on pourra fermer. Pour vraiment résoudre #4531 il faudrait : - que cette objet spipConfig soit rempli dans une fonction PHP d'abord, possiblement passant dans un nouveau pipeline dédié afin que des plugins puissent compléter en PHP des infos utiles à accéder en JS ensuite (config, traduction, etc) - qu'il soit inséré alors en JS dans cette fonction PHP aussi, appelé au tout début de insert_head et de header_prive, alors que là j'ai commencé à le remplir dans le squelette, tout simplement au même endroit que là où sont définis pour l'instant les autres variables JS de l'admin - en conséquence ces autres variables JS (à garder pour rien péter) et leur doublonnage dans spipConfig, devraient donc aussi être dans cette fonction PHP et non plus là dans ce squelette
rastapopoulos added 1 commit 1 week ago
rastapopoulos added 1 commit 1 week ago
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Assignees
Assign users
Clear assignees
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'dev/issue_4445_test_prive_js'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes