diff --git a/library/Class/Import/Typo3.php b/library/Class/Import/Typo3.php
index c9c47402dd2a81fd3d401540c4cfd5ac8571ff05..bf870e17a23e675eb4b629801ab12dea73206882 100644
--- a/library/Class/Import/Typo3.php
+++ b/library/Class/Import/Typo3.php
@@ -22,7 +22,8 @@ class Class_Import_Typo3 {
const DEFAULT_CAT_ID = 30000;
protected
- $user_mapper;
+ $user_mapper,
+ $unknown_koha_urls = [];
public function __construct() {
$this->t3db = new Typo3DB();
@@ -190,7 +191,44 @@ class Class_Import_Typo3 {
if(false !== ($end = strpos($link_array[1], '>')))
$url = substr($url, 0, $end);
- return '<a href="' . $url . '"' . $target . '>' . $this->getContentFromLink($link) . '</a>';
+ $url = $this->convertKohaToBokehUrl($url);
+
+ return '<a href="' . $url . '">' . $this->getContentFromLink($link) . '</a>';
+ }
+
+
+ protected function convertKohaToBokehUrl($url) {
+ if (preg_match('/biblionumber=(\d+)$/', $url, $matches))
+ return $this->urlViewNotice($url, $matches[1]);
+
+
+ if (preg_match('/opac-search.pl\?q=au:(.+)$/', $url, $matches))
+ return $this->urlSearchAuthors($url, $matches[1]);
+
+
+ if (preg_match('/opac-search.pl\?q=([^=&]+)$/', $url, $matches))
+ return $this->urlSimpleSearch($url, $matches[1]);
+
+ $this->unknown_koha_urls[] = $url;
+
+ return $url;
+ }
+
+ protected function urlSearchAuthors($url, $search_expression) {
+ return '/miop-test.net/recherche/simple/rech_auteurs/'.$search_expression;
+ }
+
+
+ protected function urlSimpleSearch($url, $search_expression) {
+ return '/miop-test.net/recherche/simple/expressionRecherche/'.$search_expression;
+ }
+
+ protected function urlViewNotice($url, $id_origine) {
+ if (!$exemplaire = Class_Exemplaire::findFirstBy(['id_origine' => $id_origine]))
+ return $url;
+
+ $clef_alpha = $exemplaire->getNotice()->getClefAlpha();
+ return '/miop-test.net/recherche/viewnotice/clef/'.$clef_alpha;
}
@@ -312,27 +350,10 @@ class Class_Import_Typo3 {
$pages_cat = Class_ArticleCategorie::newInstance(['libelle' => 'Pages fixes']);
$pages_cat->save();
foreach($rows as $new) {
- $date_creation = date("Y-m-d H:i:s", $new['crdate']);
- $debut = $new['starttime'] ? date("Y-m-d", $new['starttime']) : '';
- $fin = $new['endtime'] ? date("Y-m-d", $new['endtime']) : '';
-
- $element = Class_Article::newInstance(['date_creation' => $date_creation,
- 'debut' => $debut,
- 'fin' => $new['hidden'] ? null : $fin,
- 'id_user' => $this->userMap->find($new['cruser_id']),
- 'description' => '',
- //'contenu' => $new['bodytext'] ? $this->replaced_by_br($new['bodytext']) :' ',
- 'contenu' => $new['bodytext'] ? $this->manipulate_body($new['bodytext']) :'',
- 'id_cat' => $pages_cat->getId(),
- 'status' => Class_Article::STATUS_VALIDATED,
- 'tags' => '',
- 'titre' => $new['header']]);
-
- if (!$element->save()) {
- $this->report['pages_errors']++;
- $this->errors[] = 'pages with uid: ' . $new['uid'] . ' (' . $new['title'] . ') - ' . implode(', ', $element->getErrors());
- continue;
- }
+ $this->traceUnknownKohaUrls(
+ function() use ($new, $pages_cat){
+ return $this->createArticlePage($new, $pages_cat);
+ });
$this->report['pages_created']++;
}
@@ -340,6 +361,43 @@ class Class_Import_Typo3 {
}
+ public function createArticlePage($new, $pages_cat) {
+ $date_creation = date("Y-m-d H:i:s", $new['crdate']);
+ $debut = $new['starttime'] ? date("Y-m-d", $new['starttime']) : '';
+ $fin = $new['endtime'] ? date("Y-m-d", $new['endtime']) : '';
+
+
+ $element = Class_Article::newInstance(['date_creation' => $date_creation,
+ 'debut' => $debut,
+ 'fin' => $new['hidden'] ? null : $fin,
+ 'id_user' => $this->userMap->find($new['cruser_id']),
+ 'description' => '',
+ //'contenu' => $new['bodytext'] ? $this->replaced_by_br($new['bodytext']) :' ',
+ 'contenu' => $new['bodytext'] ? $this->manipulate_body($new['bodytext']) :'',
+ 'id_cat' => $pages_cat->getId(),
+ 'status' => Class_Article::STATUS_VALIDATED,
+ 'tags' => '',
+ 'titre' => $new['header']]);
+
+ if (!$element->save()) {
+ $this->report['pages_errors']++;
+ $this->errors[] = 'pages with uid: ' . $new['uid'] . ' (' . $new['title'] . ') - ' . implode(', ', $element->getErrors());
+ }
+ return $element;
+ }
+
+
+ public function traceUnknownKohaUrls($closure) {
+ $this->unknown_koha_urls = [];
+ $element = $closure();
+ foreach($this->unknown_koha_urls as $url) {
+ Class_Import_Typo3_Logs::getInstance()->addUnknownUrl($element,
+ $url);
+ }
+ return $element;
+ }
+
+
public function format_url($url) {
if (!$url = explode(' ', $url)[0])
diff --git a/library/Class/Import/Typo3/Logs.php b/library/Class/Import/Typo3/Logs.php
index 6701c1a9c8f6f2fb8b6b67d7a3f5b16e4a772479..9ef4e4cb19324f1b2159302194df5bf94c126477 100644
--- a/library/Class/Import/Typo3/Logs.php
+++ b/library/Class/Import/Typo3/Logs.php
@@ -50,6 +50,10 @@ class Class_Import_Typo3_logs {
return $this->_logs;
}
+ public function addUnknownUrl($element, $url) {
+ $this->addLogRow(get_class($element).' id:'.$element->getId().' unknown URL: '.$url);
+ }
+
public function addLogRow($log) {
$this->_logs .= $log . '\n';
diff --git a/tests/library/Class/Import/Typo3Fixture.php b/tests/library/Class/Import/Typo3Fixture.php
index c0539f0c13a5c0b8de1047f2a28d6e11d6a45aa1..41f57baf08584a45414b6ad1b770347980ff2b98 100644
--- a/tests/library/Class/Import/Typo3Fixture.php
+++ b/tests/library/Class/Import/Typo3Fixture.php
@@ -238,9 +238,9 @@ La collection<span style="text-decoration: none"><span style="font-style: normal
'category' => 1,
'tx_danpextendnews_tags' => '',
'title' => 'Realy ? you crash ?',
- 'bodytext' => '<span style="font-weight: bold;">Titre :</span> No Longer at Ease<br /><span style="font-weight: bold;">Interprètes :</span> NNEKA<br /><span style="font-weight: bold;">Compositeur :</span> NNEKA<br /><span style="font-weight: bold;">Label :</span> Yo Mamma Records<br /><span style="font-weight: bold;">Date :</span> 2008<br /><span style="font-weight: bold;">Genre :</span> Nu Soul<br /><span style="font-weight: bold;">Support :</span> CD<br /><span style="font-weight: bold;">Localisation :</span> Fos-sur-Mer, Miramas <br /><link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-detail.pl?biblionumber=174002 _top external-link-new-window><span style="font-weight: bold;">Lien direct au catalogue</span></link><br /><br />
+ 'bodytext' => '<span style="font-weight: bold;">Titre :</span> No Longer at Ease<br /><span style="font-weight: bold;">Interprètes :</span> NNEKA<br /><span style="font-weight: bold;">Compositeur :</span> NNEKA<br /><span style="font-weight: bold;">Label :</span> Yo Mamma Records<br /><span style="font-weight: bold;">Date :</span> 2008<br /><span style="font-weight: bold;">Genre :</span> Nu Soul<br /><span style="font-weight: bold;">Support :</span> CD<br /><span style="font-weight: bold;">Localisation :</span> Fos-sur-Mer, Miramas <br /><link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-detail.pl?biblionumber=174002 _top external-link-new-window>Lien direct au catalogue</link><br /><br />
<p style="width: 220px; height: 55px;"><object height="55" width="220"><param name="movie" value="http://www.deezer.com/embedded/small-widget-v2.swf?idSong=607896&colorBackground=0x555552&textColor1=0xFFFFFF&colorVolume=0x99FFFF&autoplay=0"></param><embed src="http://www.deezer.com/embedded/small-widget-v2.swf?idSong=607896&colorBackground=0x555552&textColor1=0xFFFFFF&colorVolume=0x99FFFF&autoplay=0" type="application/x-shockwave-flash" height="55" width="220"></embed></object><br />Découvrez <link http://www.deezer.com/fr/nneka.html>Nneka</link>!</p>
-<br /><br /><span style="font-weight: bold;">Notre avis :</span><br /><span style="font-style: italic;">No Longer at Ease</span> est un album qui parle de vérité, de sincérité et de lucidité sur ses propres erreurs. Nneka née au Nigéria, a grandi en Allemagne sur les sons de Fela Kuti, de Bob Marley, de Nas ou encore de Lauryn Hill.<br />Entre Reggae, Soul et électro elle incorpore aussi une pincée de rock à ses productions.<br />Le détonnant <span style="font-style: italic;">Heartbeat</span> en témoigne et délivre un message sur la guerre qui laisse le libre arbitre à chacun. <br />Si vous aimez la voix cassée de Neneh Cherry, l\'énergie de Amy Winehouse et le message universaliste de Bob Marley vous aimerez Nneka !<br /> <br /><span style="font-weight: bold;">Si vous avez aimé, vous aimerez aussi :</span><br /><link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Alicia%20Keys _blank external-link-new-window>Alicia Keys</link> - <link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Bob%20Marley%20and%20The%20Wailers _blank external-link-new-window>Bob Marley</link> - <link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Neneh%20Cherry _blank external-link-new-window>Neneh Cherry</link> - <link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Amy%20Winehouse _blank external-link-new-window>Amy Winehouse</link>'],
+<br /><br /><span style="font-weight: bold;">Notre avis :</span><br /><span style="font-style: italic;">No Longer at Ease</span> est un album qui parle de vérité, de sincérité et de lucidité sur ses propres erreurs. Nneka née au Nigéria, a grandi en Allemagne sur les sons de Fela Kuti, de Bob Marley, de Nas ou encore de Lauryn Hill.<br />Entre Reggae, Soul et électro elle incorpore aussi une pincée de rock à ses productions.<br />Le détonnant <span style="font-style: italic;">Heartbeat</span> en témoigne et délivre un message sur la guerre qui laisse le libre arbitre à chacun. <br />Si vous aimez la voix cassée de Neneh Cherry, l\'énergie de Amy Winehouse et le message universaliste de Bob Marley vous aimerez Nneka !<br /> <br /><span style="font-weight: bold;">Si vous avez aimé, vous aimerez aussi :</span><br /><link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Alicia%20Keys _blank external-link-new-window>Alicia Keys</link> - <link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Bob%20Marley%20and%20The%20Wailers _blank external-link-new-window>Bob Marley</link> - <link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Neneh%20Cherry _blank external-link-new-window>Neneh Cherry</link> - <link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=Amy%20Winehouse _blank external-link-new-window>Amy Winehouse</link>'],
['uid' => 123,
'starttime' => 0,
'endtime' => 0,
@@ -263,7 +263,7 @@ La collection<span style="text-decoration: none"><span style="font-style: normal
'tx_danpextendnews_tags' => '',
'title' => 'L\'artothèque',
'bodytext' => '<p style="color: rgb(0, 0, 0); font-family: Verdana, sans-serif; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">L\'artothèque de la Médiathèque Intercommunale propose au public sur les sites du réseau, une collection de 1 825 œuvres d\'art contemporain constituant un large panorama de la création plastique et photographique des quarante dernières années.</p>
-<p style="color: rgb(0, 0, 0); font-family: Verdana, sans-serif; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?idx=kw&idx=kw&idx=kw&limit=mc-itemtype%3AOART&sort_by=relevance&do=Rechercher _blank external-link-new-window>Un catalogue iconographique</link> est spécialement dédié à cette collection; il est également consultable dans tous les pôles <span style="font-style: italic;">Art</span>,<span style="font-style: italic;"> </span>Musique, Cinéma du réseau.<br /><br />Outre le prêt d\'œuvres d\'art, l\'artothèque organise aussi des expositions, réalise des animations, accueille des groupes, édite des catalogues..., favorisant ainsi la diffusion et la promotion de l\'art et des artistes d\'aujourd\'hui.</p>']
+<p style="color: rgb(0, 0, 0); font-family: Verdana, sans-serif; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><link http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=test&idx=kw&idx=kw&idx=kw&limit=mc-itemtype%3AOART&sort_by=relevance&do=Rechercher _blank external-link-new-window>Un catalogue iconographique</link> est spécialement dédié à cette collection; il est également consultable dans tous les pôles <span style="font-style: italic;">Art</span>,<span style="font-style: italic;"> </span>Musique, Cinéma du réseau.<br /><br />Outre le prêt d\'œuvres d\'art, l\'artothèque organise aussi des expositions, réalise des animations, accueille des groupes, édite des catalogues..., favorisant ainsi la diffusion et la promotion de l\'art et des artistes d\'aujourd\'hui.</p>']
];
}
diff --git a/tests/library/Class/Import/Typo3Test.php b/tests/library/Class/Import/Typo3Test.php
index 8cef371fd37b10aa95c836ecd54e6448648d3fe8..05ce8f385d18f542acd4bfd6fcf69733886ba3f9 100644
--- a/tests/library/Class/Import/Typo3Test.php
+++ b/tests/library/Class/Import/Typo3Test.php
@@ -181,19 +181,23 @@ class Import_Typo3CategoryTest extends Import_Typo3TestCase {
class Import_Typo3ArticleTest extends Import_Typo3TestCase {
-
-
public function setUp() {
parent::setUp();
$this->fixture('Class_ArticleCategorie', ['id' => 1,
'libelle' => 'Infos',
'sous_categories' => []]);
+ $this->fixture('Class_Exemplaire',
+ ['id' => 3,
+ 'id_origine' => 174002,
+ 'notice' => $this->fixture('Class_Notice',
+ ['id' => 5,
+ 'clef_alpha' => 'BOBMARLEY'])]);
+
$this->migration->import_user();
$this->migration->import_categories();
$this->migration->importArticles();
$this->bateau_pirate = Class_Article::findFirstBy(['titre' => 'A bord du bateau pirate']);
-
}
@@ -245,7 +249,7 @@ class Import_Typo3ArticleTest extends Import_Typo3TestCase {
/** @test */
public function bokehWikiContentShouldContainsAnchorWithExpectedAttributs() {
- $this->assertEquals('<p>Welcome to <a href="http://wiki.bokeh-library-portal.org" target="_blank">Wiki Bokeh</a></p>', Class_Article::findFirstBy(['titre' => 'Wiki Bokeh'])->getContenu());
+ $this->assertEquals('<p>Welcome to <a href="http://wiki.bokeh-library-portal.org">Wiki Bokeh</a></p>', Class_Article::findFirstBy(['titre' => 'Wiki Bokeh'])->getContenu());
}
@@ -257,8 +261,22 @@ class Import_Typo3ArticleTest extends Import_Typo3TestCase {
/** @test */
- public function contentOfRealyYouCrashShouldContainsAnchorWithExpectedUrl() {
- $this->assertContains('<a href="http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=au:Bob%20Marley%20and%20The%20Wailers" target="_blank">Bob Marley</a>', Class_Article::findFirstBy(['titre' => 'Realy ? you crash ?'])->getContenu());
+ public function contentOfRealyYouCrashShouldReplaceSearchAuthors() {
+ $this->assertContains('<a href="/miop-test.net/recherche/simple/rech_auteurs/Bob%20Marley%20and%20The%20Wailers">Bob Marley</a>', Class_Article::findFirstBy(['titre' => 'Realy ? you crash ?'])->getContenu());
+ }
+
+
+ /** @test */
+ public function contentOfRealyYouCrashShouldReplaceSimpleSearches() {
+ $this->assertContains('<a href="/miop-test.net/recherche/simple/expressionRecherche/Amy%20Winehouse">',
+ Class_Article::findFirstBy(['titre' => 'Realy ? you crash ?'])->getContenu());
+ }
+
+
+ /** @test */
+ public function contentOfRealyYouCrashShouldContainsLinkToNoticeBobMarley() {
+ $this->assertContains('<a href="/miop-test.net/recherche/viewnotice/clef/BOBMARLEY">Lien direct au catalogue</a>',
+ Class_Article::findFirstBy(['titre' => 'Realy ? you crash ?'])->getContenu());
}
@@ -276,7 +294,7 @@ class Import_Typo3ArticleTest extends Import_Typo3TestCase {
/** @test */
public function lartothequeArticleShouldBeTranformed() {
- $this->assertContains('<a href="http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?idx=kw&idx=kw&idx=kw&limit=mc-itemtype%3AOART&sort_by=relevance&do=Rechercher" target="_blank">Un catalogue iconographique</a>', Class_Article::findFirstBy(['titre' => 'L\'artothèque'])->getContenu());
+ $this->assertContains('<a href="http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-search.pl?q=test&idx=kw&idx=kw&idx=kw&limit=mc-itemtype%3AOART&sort_by=relevance&do=Rechercher">Un catalogue iconographique</a>', Class_Article::findFirstBy(['titre' => 'L\'artothèque'])->getContenu());
}
}
@@ -417,6 +435,13 @@ class Import_Typo3LogsTest extends Import_Typo3TestCase {
}
+ /** @test */
+ public function logsShouldContainsUrlKohaNotFoundForArticle23() {
+ $this->assertContains('Class_Article id:1 unknown URL: http://koha.mediathequeouestprovence.fr/cgi-bin/koha/opac-shelves.pl?viewshelf=1556&sortfield=title%20%3Ccid:part1.08080603.08080008@ouestprovence.fr',
+ Class_Import_Typo3_Logs::getInstance()->getLogs());
+ }
+
+
/** @test */
public function lgosShouldReturn3UsersAdded() {
$this->assertContains('User(s) added: 3', Class_Import_Typo3_Logs::getInstance()->getLogs());