Browse Source

r97980 avait remis les ul/li sur le formulaire au lieu des .editer-groupe/.editer, et la lecture de la config par lire_config pour une simple meta n'est pas optimale, surtout dans public/stats appele a chaque hit

spip-3.1
cedric@yterium.com 7 years ago
parent
commit
241d1fc2af
  1. 26
      formulaires/configurer_compteur.html
  2. 2
      genie/visites.php
  3. 2
      paquet.xml
  4. 3
      public/stats.php
  5. 4
      stats_autoriser.php

26
formulaires/configurer_compteur.html

@ -5,31 +5,31 @@
[<p class="reponse_formulaire reponse_formulaire_erreur">(#ENV*{message_erreur})</p>]
<form action="#ENV{action}#formulaire_configurer_compteur" method="post"><div>
[(#ACTION_FORMULAIRE{#ENV{action}})]
<ul>
<li class='editer configurer_activer_statistiques[ (#ENV**{erreurs}|table_valeur{activer_statistiques}|oui)erreur]'>
<div class="editer-groupe">
<div class='editer configurer_activer_statistiques[ (#ENV**{erreurs}|table_valeur{activer_statistiques}|oui)erreur]'>
[<span class='erreur_message'>(#ENV**{erreurs}|table_valeur{activer_statistiques})</span>]
<div class='choix'>
<input class='checkbox' type="checkbox" name="activer_statistiques" value='oui' [checked="(#ENV{activer_statistiques}|=={oui}|?{checked})"] id="activer_statistiques"/>
<label for="activer_statistiques"><:statistiques:item_gerer_statistiques:></label>
</div>
</li>
<li class='editer configurer_activer_referers[ (#ENV**{erreurs}|table_valeur{activer_referers}|oui)erreur]'>
</div>
<div class='editer configurer_activer_referers[ (#ENV**{erreurs}|table_valeur{activer_referers}|oui)erreur]'>
[<span class='erreur_message'>(#ENV**{erreurs}|table_valeur{activer_referers})</span>]
<div class='choix'>
<input class='checkbox' type="checkbox" name="activer_referers" value='oui' [checked="(#ENV{activer_referers, oui}|=={oui}|?{checked})"] id="activer_referers"/>
<label for="activer_referers"><:statistiques:info_referer_oui:></label>
</div>
</li>
<li class='editer configurer_activer_captures_referers[ (#ENV**{erreurs}|table_valeur{activer_captures_referers}|oui)erreur]'>
</div>
<div class='editer configurer_activer_captures_referers[ (#ENV**{erreurs}|table_valeur{activer_captures_referers}|oui)erreur]'>
<div class='explication'><:statistiques:info_question_vignettes_referer:></div>
[<span class='erreur_message'>(#ENV**{erreurs}|table_valeur{activer_captures_referers})</span>]
<div class='choix'>
<input class='checkbox' type="checkbox" name="activer_captures_referers" value='oui' [checked="(#ENV{activer_captures_referers}|=={oui}|?{checked})"] id="activer_captures_referers"/>
<label for="activer_captures_referers"><:statistiques:info_question_vignettes_referer_oui:></label>
</div>
</li>
</div>
</ul>
</div>
<p class='boutons'><input class='submit' type="submit" name="ok" value="<:bouton_enregistrer:>"/></p>
</div></form>
@ -37,15 +37,15 @@
<script type='text/javascript'>
function actualise_referers(){
if (jQuery('#activer_statistiques').prop('checked')) {
jQuery('#formulaire_configurer_compteur li.configurer_activer_captures_referers').show();
jQuery('#formulaire_configurer_compteur .editer.configurer_activer_captures_referers').show();
if (jQuery('#activer_referers').prop('checked'))
jQuery('#formulaire_configurer_compteur li.configurer_activer_referers').show();
jQuery('#formulaire_configurer_compteur .editer.configurer_activer_referers').show();
else
jQuery('#formulaire_configurer_compteur li.configurer_activer_captures_referers').hide();
jQuery('#formulaire_configurer_compteur .editer.configurer_activer_captures_referers').hide();
}
else {
jQuery('#formulaire_configurer_compteur li.configurer_activer_captures_referers').hide();
jQuery('#formulaire_configurer_compteur li.configurer_activer_referers').hide();
jQuery('#formulaire_configurer_compteur .editer.configurer_activer_captures_referers').hide();
jQuery('#formulaire_configurer_compteur .editer.configurer_activer_referers').hide();
}
}
jQuery(document).ready(function(){

2
genie/visites.php

@ -210,7 +210,7 @@ function calculer_visites($t) {
## Ajouter un JOIN sur le statut de l'article ?
}
}
if (lire_config('activer_referers', 'oui') == "oui") {
if (!isset($GLOBALS['meta']['activer_referers']) or $GLOBALS['meta']['activer_referers'] == "oui") {
// 3. Les referers du site
// insertion pour les nouveaux, au tableau des increments sinon
if ($referers[$date]) {

2
paquet.xml

@ -1,7 +1,7 @@
<paquet
prefix="stats"
categorie="statistique"
version="1.0.4"
version="1.0.5"
etat="stable"
compatibilite="[3.1.0;3.1.*]"
logo="prive/themes/spip/images/statistique-32.png"

3
public/stats.php

@ -85,8 +85,7 @@ function public_stats_dist($contexte = null, $referer = null) {
// Analyse du referer
$log_referer = '';
include_spip('inc/config');
if (lire_config('activer_referers', 'oui') == "oui") {
if (!isset($GLOBALS['meta']['activer_referers']) or $GLOBALS['meta']['activer_referers'] == "oui") {
if (isset($referer)) {
$url_site_spip = preg_replace(',/$,', '',
preg_replace(',^(https?://)?(www\.)?,i', '',

4
stats_autoriser.php

@ -46,7 +46,7 @@ function autoriser_statistiques_menu_dist($faire, $type = '', $id = 0, $qui = nu
* @return bool true s'il a le droit, false sinon
*/
function autoriser_referers_menu_dist($faire, $type = '', $id = 0, $qui = null, $opt = null) {
return (lire_config('activer_referers', 'oui') == "oui") && autoriser('voirstats', $type, $id, $qui, $opt);
return (!isset($GLOBALS['meta']['activer_referers']) or $GLOBALS['meta']['activer_referers'] == "oui") && autoriser('voirstats', $type, $id, $qui, $opt);
}
@ -131,5 +131,5 @@ function autoriser_statslang_onglet_dist($faire, $type, $id, $qui, $opt) {
* @return bool true s'il a le droit, false sinon
*/
function autoriser_statsreferers_onglet_dist($faire, $type, $id, $qui, $opt) {
return (lire_config('activer_referers', 'oui') == "oui") && autoriser('voirstats', $type, $id, $qui, $opt);
return (!isset($GLOBALS['meta']['activer_referers']) or $GLOBALS['meta']['activer_referers'] == "oui") && autoriser('voirstats', $type, $id, $qui, $opt);
}

Loading…
Cancel
Save