Pods: Problème réel : les noms réservés ne doivent pas être ajoutés

Issue-Label Bot applique automatiquement l'étiquette Type: Bug à ce problème, avec une confiance de 0,98. Veuillez marquer ce commentaire avec :thumbsup: ou :thumbsdown: pour donner votre avis à notre bot !

Liens : page d'accueil de l'application , tableau de bord et code pour ce bot.

+1 J'ai le même problème. La définition de noms préservés ne génère pas d'erreur.

Voir #4122 Ceci devrait être appliqué aux noms de champ, taxonomie, post-types, etc. Quelqu'un ne devrait pas être en mesure de taper AUCUN nom de champ, post-type ou taxonomie qui se trouve sur la liste de champs de noms RÉSERVÉS.

J'ai déposé un rapport de bogue #5440 où j'ai eu le même problème avec un champ intitulé "statut". Cependant, « statut » ne figure pas sur la liste des termes réservés de Wordpress. Dans le cas mentionné ci-dessus, « nom » figure sur cette liste, mais pas « foobar ». Il en est de même pour la liste PHP des mots réservés.

Liste Wordpress : (https://codex.wordpress.org/Reserved_Terms)
Liste PHP https://www.php.net/manual/en/reserved.php

Peut-être qu'une autre liste de mots réservés interfère aussi ?

@raoulunger Avez-vous essayé les dernières RP concernant les mots-clés réservés ?

EDIT : status n'est pas un mot-clé réservé à WordPress, mais il l'est pour les Pods car il est utilisé comme l'un des champs d'objet Pod. (encore enquête en ce moment)

Salut, je vois que tu as rayé la question ;-). Merci d'avoir enquêté !

Oui, après enquête, il s'avère que status est un alias pour post_status dans les Pods (à utiliser dans les Magic Tags, etc.).
Liste complète des clés et alias par défaut :

ID                     ==> id
post_title             ==> title & name
post_content           ==> content
post_excerpt           ==> excerpt
post_author            ==> author
post_date              ==> created & date
post_date_gmt
post_status            ==> status
comment_status
ping_status
post_password
post_name              ==> slug &  permalink
to_ping
pinged
post_modified          ==> modified
post_modified_gmt
post_content_filtered
post_parent            ==> parent
guid
menu_order
post_type              ==> type
post_mime_type
comment_count
comments

@sc0ttkclark @jimtrue Je pense que ce problème peut être clos, d'accord ?

D'accord, cela clarifie les choses en ce qui concerne le « statut » – merci d'avoir compris cela, et pour la liste !

Pour résoudre le problème plus large de l'utilisation de mots réservés (ce sujet, ainsi que #4122), il semble qu'il y aurait plusieurs options :

• le code est modifié afin que les alias pour les identifiants puissent en fait être utilisés, ou (si ce n'est pas faisable),
• un avertissement s'affiche lorsqu'un mot 'interdit' est utilisé, et
• une référence est faite quelque part dans la documentation à ce phénomène et à la liste ci-dessus pour les mots interdits.

Au fait, il m'est facile de suggérer tout cela bien sûr, car je ne peux pas vraiment vous aider ici - mes compétences en codage sont beaucoup trop limitées, désolé !

Acclamations!

L'ID est simplement un mot-clé réservé, je ne pense pas que nous puissions changer cela.
À propos des avertissements, ils sont actuellement affichés, mais cela se fait via AJAX (côté serveur) et non directement lors de la frappe. Je pense que c'est quelque chose que nous ajouterons à l'avenir, mais pas en tant que version de maintenance.
La documentation est toujours une bonne idée. @jimtrue Pourriez-vous créer une page de documentation pour cela que je puisse utiliser ?

Ne peut pas être fermé tant qu'il n'est pas documenté. Y a-t-il un autre de ces « alias » qui ne fonctionnerait pas pour la création en tant que champ ? c'est-à-dire « titre », « parent », etc ?

C'est tout ce que j'ai pu trouver jusqu'à présent (à part les mots-clés de base WP déjà mentionnés)

Cela aurait encore besoin d'être corrigé car nous devons expliquer POURQUOI nous n'autorisons pas l'ajout de ces champs. Pour le moment, nous ne leur donnons aucune information.

Ce n'est pas tout à fait vrai, AJAX renvoie un message d'erreur, mais uniquement lors de l'enregistrement.

Je n'ai reçu aucune erreur visible lorsque j'ai dupliqué le problème. S'il n'y a pas d'erreur visible pour l'utilisateur, c'est un problème.

Avez-vous testé avec le dernier PR ? #5441
Gardez à l'esprit que ces mots-clés réservés sont autorisés pour les noms de champ. Mais pas pour les Pods (CPT/Tax), voir #5428.

Je n'ai pas non plus reçu de message d'erreur (d'où la confusion). Je n'ai pas testé avec le dernier PR, car je n'ai vraiment aucune idée de comment faire cela (désolé encore - je suis assez novice ici).

Le 19 août 2019, à 17h50, Jory Hogeveen [email protected] a écrit :

Avez-vous testé avec le dernier PR ? #5443 https://github.com/pods-framework/pods/pull/5443
Gardez à l'esprit que ces mots-clés réservés sont autorisés pour les noms de champs. Mais pas pour les Pods (CPT/Tax), voir #5428 https://github.com/pods-framework/pods/pull/5428 .

-
Vous recevez ceci parce que vous avez été mentionné.
Répondre à cet e - mail directement, voir sur GitHub https://github.com/pods-framework/pods/issues/5420?email_source=notifications&email_token=AM2NSKDD43ZU2JY47LPKRV3QFK6LTA5CNFSM4ICZDB72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4TNDNQ#issuecomment-522637750 , ou couper le fil https://github.com/ notifications/unsubscribe-auth/AM2NSKESXEIVUY3PLHRRMW3QFK6LTANCNFSM4ICZDB7Q .

@sc0ttkclark @jimtrue Cela peut-il être fermé ?

Si nous déplaçons le vidage du cache avant d'envoyer le pods_error, nous pouvons résoudre le focus principal de ce ticket.

Corrigé dans #5459

Désolé, réaction tardive, mais merci !

Le 5 septembre 2019, à 14h27, Scott Kingsley Clark [email protected] a écrit :

Fermé #5420 https://github.com/pods-framework/pods/issues/5420 via #5459 https://github.com/pods-framework/pods/pull/5459 .

-
Vous recevez ceci parce que vous avez été mentionné.
Répondre à cet e - mail directement, voir sur GitHub https://github.com/pods-framework/pods/issues/5420?email_source=notifications&email_token=AM2NSKBMJPPMDU5DSLKEPWTQID3JNA5CNFSM4ICZDB72YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTOL3AQY#event-2610409539 , ou couper le fil https://github.com/ notifications/unsubscribe-auth/AM2NSKDMLL2WOT3F7ITJ47TQID3JNANCNFSM4ICZDB7Q .