hotline : non display draft articles
Merged
requested to merge hotline#116263_magasin_de_themes_bug_articles_brouillons_visibles_en_non_connecte into hotline
3 unresolved threads
Merge request reports
Activity
Filter activity
27 27 28 28 protected function _findElements() { 29 29 $loader = Class_Article::getLoader(); 30 $this->preferences['size'] = 100; 31 $this->_articles = $loader->getArticlesByPreferences($this->preferences); 30 $preferences = $this->preferences; 31 if (isset($this->preferences['preferences'])) 32 $preferences = $this->preferences['preferences']; 33 $preferences['size'] = 100; 34 35 if (Class_AdminVar::isWorkflowEnabled() changed this line in version 2 of the diff
158 'id_categorie' =>'', 159 'events_only' => true, 160 'event_date' => "", 161 'id_bib' => 0, 162 'id_lieu' =>'', 163 'custom_fields' => [], 164 'published' => true, 165 'event_start_after' => '2020-09', 166 'event_end_after' => '', 167 'limit' => 9 168 ]) 169 ->answers([]) 170 171 ->beStrict(); 172 $this->dispatch('/opac/index/index/id_profil/1'); 173 $this->assertNotXPathContentContains('//div', 'Un super article'); changed this line in version 7 of the diff
300 406 /** @test */ 301 407 public function selectRecordShouldBePresent() { 302 408 Class_AdminVar::set('ENABLE_SEARCH_MULTIPLE_RECORD_SELECTION', 1); 303 $this->dispatch('/recherche/simple/expressionRecherche/jardin'); 409 $this->dispatch('/recherche/simple/expressionRecherche/jardin'); changed this line in version 3 of the diff
Ce fix ne me convient pas vraiment , le filterByLocaleAndWorkflow devrait suffir ( pas besoin de faire une requete avec les parametres status et langue) , cependant si on ne le fait pas au moment de la requete le limit ne fonctionnera pas (le limit ne peut renvoyer que des articles avec un statut brouillon ou d'une langue differente)... dans d'autres parties du code on ne fait pas le filtrage au niveau SQL (par ex : Class_Systeme_ModulesMenu_News)je suis bien incapable de dire si cela pose un soucis ou non...
added 1 commit
- f13c379f - hotline #116263 templates : fix workflow usage for articles
enabled an automatic merge when the pipeline for f13c379f succeeds
mentioned in commit daf0d096