diff --git a/VERSIONS_HOTLINE/37150 b/VERSIONS_HOTLINE/37150
new file mode 100644
index 0000000000000000000000000000000000000000..f0cf5d2b79574406776776683a1d8f31a6e9344f
--- /dev/null
+++ b/VERSIONS_HOTLINE/37150
@@ -0,0 +1 @@
+ - ticket #37150 : Amélioration de la prise en compte des facettes dans l'affichage d'une notice
diff --git a/library/Class/Notice.php b/library/Class/Notice.php
index 409b66b4771694c664dab897813fcd0a25574061..1f8fa8875035383a1157b4f59b64448cd8c47b9e 100644
--- a/library/Class/Notice.php
+++ b/library/Class/Notice.php
@@ -669,10 +669,10 @@ class Class_Notice extends Storm_Model_Abstract {
$codification = Class_Codification::getInstance();
$items = array_filter(explode(' ', trim($facettes)));
foreach ($items as $item) {
- if ($item[0] != $champ)
+ if (($item[0] != $champ) || (!$label = $codification->getLibelleFacette($item)))
continue;
- $ret[] = $this->getReboundFieldForFacet($item, $codification->getLibelleFacette($item));
+ $ret[] = $this->getReboundFieldForFacet($item, $label);
}
return $ret;
}
diff --git a/tests/library/Class/NoticeTest.php b/tests/library/Class/NoticeTest.php
index 48b267d1bdc3e7e79e624a9ee2783af41c9a1b39..d56709d737b4a57cbb945fc71e6b1dc25945f870 100644
--- a/tests/library/Class/NoticeTest.php
+++ b/tests/library/Class/NoticeTest.php
@@ -134,7 +134,7 @@ class NoticeVignetteTest extends Storm_Test_ModelTestCase {
/** @test */
public function withoutThumbnailFetchUrlVignetteShouldAnswerVignetteActionForAjax() {
$this->assertContains('/recherche/vignette/clef/',
- $this->_notice_without_thumbnails->fetchUrlVignette());
+ $this->_notice_without_thumbnails->fetchUrlVignette());
$this->assertFalse($this->_http_client->methodHasBeenCalled('open_url'));
}
@@ -420,11 +420,11 @@ class NoticeTestGetAvis extends ModelTestCase {
'abon_ou_bib' => 0,
'note' => 3]));
$this->onLoaderOfModel('Class_AvisNotice')
- ->whenCalled('findAllBy')
- ->with(['clef_oeuvre' => 'TESTING-RECORD--'])
- ->answers([$this->avis_bib1, $this->avis_bib2,
- $this->avis_abon1, $this->avis_abon2])
- ->beStrict();
+ ->whenCalled('findAllBy')
+ ->with(['clef_oeuvre' => 'TESTING-RECORD--'])
+ ->answers([$this->avis_bib1, $this->avis_bib2,
+ $this->avis_abon1, $this->avis_abon2])
+ ->beStrict();
$this->notice = $this->fixture('Class_Notice',
['id' => 12,
@@ -513,21 +513,21 @@ abstract class NoticeThirdPartyTracksTestCase extends Storm_Test_ModelTestCase {
public function setUp() {
parent::setUp();
- parent::setUp();
+ parent::setUp();
$zone_995 = serialize(
- [
- ['clef' => 'h', 'valeur' => '00510172'],
- ['clef' => '2', 'valeur' => '[AA][Adulte CEC][0][1][Disponible][0][0][0][0]']]);
+ [
+ ['clef' => 'h', 'valeur' => '00510172'],
+ ['clef' => '2', 'valeur' => '[AA][Adulte CEC][0][1][Disponible][0][0][0][0]']]);
$exemplaire = $this->fixture('Class_Exemplaire',
['id' => 234,
'zone995' => $zone_995]);
$notice = $this->fixture('Class_Notice',
- ['id' => 34,
- 'type_doc' => Class_TypeDoc::DISQUE,
- 'exemplaires' => [$exemplaire],
- 'unimarc' => $this->_unimarc]);
+ ['id' => 34,
+ 'type_doc' => Class_TypeDoc::DISQUE,
+ 'exemplaires' => [$exemplaire],
+ 'unimarc' => $this->_unimarc]);
$this->_result = $notice->getMorceaux();
$this->_tracks = $this->_result['morceaux'];
@@ -650,9 +650,9 @@ class NoticeStromaeTest extends Storm_Test_ModelTestCase {
public function setUp() {
parent::setUp();
$zone_995 = serialize(
- [
- ['clef' => 'h', 'valeur' => '00510172'],
- ['clef' => '2', 'valeur' => '[AA][Adulte CEC][0][1][Disponible][0][0][0][0]']]);
+ [
+ ['clef' => 'h', 'valeur' => '00510172'],
+ ['clef' => '2', 'valeur' => '[AA][Adulte CEC][0][1][Disponible][0][0][0][0]']]);
$exemplaire = $this->fixture('Class_Exemplaire',
['id' => 234,
@@ -785,7 +785,7 @@ class NoticeStromaeTest extends Storm_Test_ModelTestCase {
class NoticeWithSerialArticleTest extends AbstractControllerTestCase {
- protected $_articles;
+ protected $_articles;
public function setUp() {
parent::setUp();
@@ -798,16 +798,16 @@ class NoticeWithSerialArticleTest extends AbstractControllerTestCase {
$this->fixture('Class_Notice_SerialArticles', ['id' => 19015,
- 'id_article' => 19015,
- 'clef_chapeau' => 'CAHIERS PEDAGOGIQUES',
- 'clef_numero' => 478,
- 'unimarc' => "00901naa0 2200169 450 001000800000100004100008200010800049215001100157330036100168600004600529606003300575610003400608610002200642700001900664701001800683801003000701 2660662 a20100127a2010 uu|y0frey0103 ba | aInventer une nouvelle maïeutique pour apprendre à apprendre bpériodique fPatrice Bride gNicole Priou app.2-4 aFrançois Taddéi est l'auteur d'un rapport pour l'OCDE, \"Former des constructeurs de savoirs créatifs et collaboratifs\". Nous l'avons rencontré au Centre de recherches interdisciplinaires, lieu symbolique pour ce chercheur au parcours à la diversité atypique en France, actuellement directeur de recherche à l'Inserm à l'université Paris Descartes. 1 aTaddéi bFrançois f19..-.... xEntretiens | aEducation xRecherche yFrance | aTaddéi, François einterview | aécole erecherche 1 aBride bPatrice 1 aPriou bNicole 0 aFR bIndexpresse c20100127 "]);
+ 'id_article' => 19015,
+ 'clef_chapeau' => 'CAHIERS PEDAGOGIQUES',
+ 'clef_numero' => 478,
+ 'unimarc' => "00901naa0 2200169 450 001000800000100004100008200010800049215001100157330036100168600004600529606003300575610003400608610002200642700001900664701001800683801003000701 2660662 a20100127a2010 uu|y0frey0103 ba | aInventer une nouvelle maïeutique pour apprendre à apprendre bpériodique fPatrice Bride gNicole Priou app.2-4 aFrançois Taddéi est l'auteur d'un rapport pour l'OCDE, \"Former des constructeurs de savoirs créatifs et collaboratifs\". Nous l'avons rencontré au Centre de recherches interdisciplinaires, lieu symbolique pour ce chercheur au parcours à la diversité atypique en France, actuellement directeur de recherche à l'Inserm à l'université Paris Descartes. 1 aTaddéi bFrançois f19..-.... xEntretiens | aEducation xRecherche yFrance | aTaddéi, François einterview | aécole erecherche 1 aBride bPatrice 1 aPriou bNicole 0 aFR bIndexpresse c20100127 "]);
$this->fixture('Class_Notice_SerialArticles', ['id' => 19016,
- 'id_article' => 19016,
- 'clef_chapeau' => 'CAHIERS PEDAGOGIQUES',
- 'clef_numero' => 478,
- 'unimarc' => "00851naa0 2200145 450 001000800000100004100008200007400049215001100123330037800134606002700512606005400539610006100593700002100654801003000675 2660665 a20100127a2010 uu|y0frey0103 ba | aEducation, arme géostratégique ? (L') bpériodique fPascal Bouchard app.5-6 aTravailler ensemble pour réussir durablement : tel était le thème du premier Wise (World Innovation Summit for Education), initié par la Qatar Foundation sous le patronage de la sheikha Mozah Bint Nasser Al Missned, qui a réuni à Doha, au Qatar, du 16 au 18 novembre 2009 un millier de leaders d'opinion influents, de décideurs et d'universitaires venus de 120 pays. | aEducation xInnovations | aCongrès et conférences yQatar yAl-Dawha (Qatar) | aéducation einnovation econgrès eAl-Dawha (Qatar) e2009 1 aBouchard bPascal 0 aFR bIndexpresse c20100127 "]);
+ 'id_article' => 19016,
+ 'clef_chapeau' => 'CAHIERS PEDAGOGIQUES',
+ 'clef_numero' => 478,
+ 'unimarc' => "00851naa0 2200145 450 001000800000100004100008200007400049215001100123330037800134606002700512606005400539610006100593700002100654801003000675 2660665 a20100127a2010 uu|y0frey0103 ba | aEducation, arme géostratégique ? (L') bpériodique fPascal Bouchard app.5-6 aTravailler ensemble pour réussir durablement : tel était le thème du premier Wise (World Innovation Summit for Education), initié par la Qatar Foundation sous le patronage de la sheikha Mozah Bint Nasser Al Missned, qui a réuni à Doha, au Qatar, du 16 au 18 novembre 2009 un millier de leaders d'opinion influents, de décideurs et d'universitaires venus de 120 pays. | aEducation xInnovations | aCongrès et conférences yQatar yAl-Dawha (Qatar) | aéducation einnovation econgrès eAl-Dawha (Qatar) e2009 1 aBouchard bPascal 0 aFR bIndexpresse c20100127 "]);
$this->_articles = Class_Notice::find(66616)->getArticlesPeriodique(66616);
}
@@ -843,4 +843,32 @@ class NoticeWithSerialArticleTest extends AbstractControllerTestCase {
}
}
-?>
\ No newline at end of file
+
+
+class NoticeGetChampNoticeTest extends Storm_Test_ModelTestCase {
+ protected $_storm_default_to_volatile = true,
+ $_record,
+ $_facets;
+
+ public function setUp() {
+ parent::setUp();
+ $this->_facets = 'A34561 A34562 Lfre propre Municipale d\'Armenti?res en Weppes bidOth?que DE FRELINGHIEN WATTRELOS T1 B13 Y1 V13 B14 Y2 V14 B9 B7 V7 B139 S69 YFonds propre B99 YMED B15 YM?diath?que Municipale d\'Armenti?res B89 S49 YBIBLIOTHEQUE B24 YEnneti?res en Weppes B104 YLa bidOth?que B36 B130 B30 YM?DIATH?QUE DE FRELINGHIEN B133 YBIBLIOTHEQUE DE WATTRELOS B79';
+
+ $this->_record = $this->fixture('Class_Notice',
+ ['id' => 55,
+ 'titres' => 'My facets are false but i can endle it.']);
+ }
+
+
+ /** @test */
+ public function hobbiesShouldBeEmpty() {
+ $this->assertEmpty($this->_record->getChampNotice(Class_CodifCentreInteret::CODE_FACETTE, $this->_facets));
+ }
+
+
+ /** @test */
+ public function subjectShouldBeEmpty() {
+ $this->assertEmpty($this->_record->getChampNotice(Class_CodifMatiere::CODE_FACETTE, $this->_facets));
+ }
+
+}
\ No newline at end of file