Skip to content
Snippets Groups Projects

Hotline#23865 reactivate visu count in stats notices

Merged Hotline#23865 reactivate visu count in stats notices
hotline#23865-reactivate-visu-count-in-stats-notices into stable
Merged Patrick Barroca requested to merge hotline#23865-reactivate-visu-count-in-stats-notices into stable

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Laurent
    Laurent @llaffont started a thread on the diff
    Last updated by Patrick Barroca
    library/Class/StatsNotices.php
    41 41 protected function _addStat($id_notice, $field) {
    42 42 $annee = date('Y');
    43 43 $mois = date('m');
    44 sqlExecute(sprintf('update notices set %s = %s + 1 where id_notice=%s',
    45 $field, $field, $id_notice));
    44
    45 // @see http://forge.afi-sa.fr/issues/23151
    46 /*
    47 sqlExecute(sprintf('update notices set %s = %s + 1 where id_notice=%s', $field, $field, $id_notice));
    • Laurent
      Laurent @llaffont ·
      Owner

      Ben j'comprends pas, on doit réactiver des compteurs en commentant une requête ?

    • Patrick Barroca
      Patrick Barroca @pbarroca ·
      Author Owner

      La précédente façon de désactiver le comptage a été de retourner true sur _isBlackListed, ce qui a eut pour effet de désactiver tout appel à _addStat. Cependant, dans _addStat il y a marquage de la notice (on veut éviter pour ne pas lock la table notice) et aggrégation de compteur mensuels dans la table stats_notices (ce qui est demandé par les client).

  • Patrick Barroca mentioned in commit e21bc6

    mentioned in commit e21bc6

Please register or sign in to reply