Knah-Tsaeb/MyShaarli
Knah-Tsaeb/MyShaarli
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Archive.org plugin: do not propose archival of private notes

Browse source 
Fixes #637
This commit is contained in:
Teromene 2016-10-11 16:37:42 +01:00
parent dc8e03bfc4
commit 5e148f8a52
2 changed files with 101 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
plugins/archiveorg/archiveorg.php
View file

@ -17,6 +17,9 @@ function hook_archiveorg_render_linklist($data)
$archive_html = file_get_contents(PluginManager::$PLUGINS_PATH . '/archiveorg/archiveorg.html'); $archive_html = file_get_contents(PluginManager::$PLUGINS_PATH . '/archiveorg/archiveorg.html');
foreach ($data['links'] as &$value) { foreach ($data['links'] as &$value) {
if($value['private'] && preg_match('/^\?[a-zA-Z0-9-_@]{6}($|&|#)/', $value['real_url'])) {
continue;
}
$archive = sprintf($archive_html, $value['url']); $archive = sprintf($archive_html, $value['url']);
$value['link_plugin'][] = $archive; $value['link_plugin'][] = $archive;
} }

102
tests/plugins/PluginArchiveorgTest.php
View file

@ -7,8 +7,8 @@
require_once 'plugins/archiveorg/archiveorg.php'; require_once 'plugins/archiveorg/archiveorg.php';
/** /**
* Class PlugQrcodeTest * Class PluginArchiveorgTest
* Unit test for the QR-Code plugin * Unit test for the archiveorg plugin
*/ */
class PluginArchiveorgTest extends PHPUnit_Framework_TestCase class PluginArchiveorgTest extends PHPUnit_Framework_TestCase
{ {
@ -21,21 +21,25 @@ function setUp()
} }
/** /**
* Test render_linklist hook. * Test render_linklist hook on external links.
*/ */
function testArchiveorgLinklist() function testArchiveorgLinklistOnExternalLinks()
{ {
$str = 'http://randomstr.com/test'; $str = 'http://randomstr.com/test';
$data = array( $data = array(
'title' => $str, 'title' => $str,
'links' => array( 'links' => array(
array( array(
'url' => $str, 'url' => $str,
'private' => 0,
'real_url' => $str
) )
) )
); );
$data = hook_archiveorg_render_linklist($data); $data = hook_archiveorg_render_linklist($data);
$link = $data['links'][0]; $link = $data['links'][0];
// data shouldn't be altered // data shouldn't be altered
$this->assertEquals($str, $data['title']); $this->assertEquals($str, $data['title']);
@ -44,5 +48,95 @@ function testArchiveorgLinklist()
// plugin data // plugin data
$this->assertEquals(1, count($link['link_plugin'])); $this->assertEquals(1, count($link['link_plugin']));
$this->assertNotFalse(strpos($link['link_plugin'][0], $str)); $this->assertNotFalse(strpos($link['link_plugin'][0], $str));
} }
/**
* Test render_linklist hook on internal links.
*/
function testArchiveorgLinklistOnInternalLinks()
{
$internalLink1 = 'http://shaarli.shaarli/?qvMAqg';
$internalLinkRealURL1 = '?qvMAqg';
$internalLink2 = 'http://shaarli.shaarli/?2_7zww';
$internalLinkRealURL2 = '?2_7zww';
$internalLink3 = 'http://shaarli.shaarli/?z7u-_Q';
$internalLinkRealURL3 = '?z7u-_Q';
$data = array(
'title' => $internalLink1,
'links' => array(
array(
'url' => $internalLink1,
'private' => 0,
'real_url' => $internalLinkRealURL1
),
array(
'url' => $internalLink1,
'private' => 1,
'real_url' => $internalLinkRealURL1
),
array(
'url' => $internalLink2,
'private' => 0,
'real_url' => $internalLinkRealURL2
),
array(
'url' => $internalLink2,
'private' => 1,
'real_url' => $internalLinkRealURL2
),
array(
'url' => $internalLink3,
'private' => 0,
'real_url' => $internalLinkRealURL3
),
array(
'url' => $internalLink3,
'private' => 1,
'real_url' => $internalLinkRealURL3
)
)
);
$data = hook_archiveorg_render_linklist($data);
// Case n°1: first link type, public
$link = $data['links'][0];
$this->assertEquals(1, count($link['link_plugin']));
$this->assertNotFalse(strpos($link['link_plugin'][0], $internalLink1));
// Case n°2: first link type, private
$link = $data['links'][1];
$this->assertArrayNotHasKey('link_plugin', $link);
// Case n°3: second link type, public
$link = $data['links'][2];
$this->assertEquals(1, count($link['link_plugin']));
$this->assertNotFalse(strpos($link['link_plugin'][0], $internalLink2));
// Case n°4: second link type, private
$link = $data['links'][3];
$this->assertArrayNotHasKey('link_plugin', $link);
// Case n°5: third link type, public
$link = $data['links'][4];
$this->assertEquals(1, count($link['link_plugin']));
$this->assertNotFalse(strpos($link['link_plugin'][0], $internalLink3));
// Case n°6: third link type, private
$link = $data['links'][5];
$this->assertArrayNotHasKey('link_plugin', $link);
}
} }