Add FacetWP services #372
Add FacetWP services #372
Conversation
ecorica
requested review from
petitphp,
francoistibo,
asadowski10,
firestar300,
cedric07,
MarieComet,
n-langle,
lphoumpakka,
stephane-gillot,
ptesei,
iazema and
a team
|
@MarieComet Rajouter le code qui permet de relier proprement les champs et les labels (reprendre ce qui a été fait sur Arendt) |
…de facets by names
inc/Services/Facet_WP.php
Outdated
| // Return default HTML if these facets are empty | ||
| $excluded_empty_facets = [ | ||
| 'checkboxes', | ||
| 'pager', | ||
| 'reset', | ||
| ]; | ||
| if ( true === in_array( $args['facet']['type'], $excluded_empty_facets, true ) && empty( $args['values'] ) ) { | ||
| return $html; | ||
| } |
There was a problem hiding this comment.
J'aurais déplacé ça au début de la fonction, juste après la partie $excluded_facet_names.
Et j'aurais créé une variable $facet_type = $args['facet']['type']; au début pour l'utiliser dans la condition
There was a problem hiding this comment.
J'aurais déplacé ça au début de la fonction, juste après la partie $excluded_facet_names.
Alors non car juste au dessus on teste la facette per_page, qui est de type pager et qui est toujours vide, mais que l'on veut quand même traiter, donc ne pas rentrer dans la condition.
On garde le type "pager" dans $excluded_empty_facets car cela correspond aussi aux facettes de type pagination.
inc/Services/Facet_WP.php
Outdated
| } | ||
|
|
||
| // Return default HTML if the facet is excluded by its name | ||
| $excluded_facet_names = []; |
There was a problem hiding this comment.
Il faudrait ajouter un filtre pour pouvoir modifier les infos depuis les plugins par exemple.
Idem pour la variable $excluded_empty_facets
| return $facets; | ||
| } | ||
|
|
||
| $facet_content = \json_decode( $facet_content, true ); |
There was a problem hiding this comment.
Attention, on a deux appels à json_decode. Il faut retirer celui-ci.
L'ajout du service FWP avec les méthodes que nous utilisons sur tous nos projets
Reste à faire : MAJ le fichier pot + po + Mo
CF : BeAPI/bea-plugin-defaults#21