From b6c0c1c9de8ffc2e6058eb2f480851da096c8aea Mon Sep 17 00:00:00 2001 From: Robert Spencer Date: Fri, 5 Jul 2013 00:07:15 +0200 Subject: [PATCH 1/9] Fix grammar, punctuation and spelling --- README.md | 4 +- inc/shaarli.css | 10 +- index.php | 276 ++++++++++++++++++++++----------------------- tpl/configure.html | 4 +- tpl/import.html | 4 +- 5 files changed, 149 insertions(+), 149 deletions(-) diff --git a/README.md b/README.md index 8f3c479b4..a27e9ab42 100644 --- a/README.md +++ b/README.md @@ -13,12 +13,12 @@ Features: * FAST * Dead-simple installation: Drop the files, open the page. No database required. * Easy to use: Single button in your browser to bookmark a page - * Save url, title, description (unlimited size). Classify links with tags (with autocomplete) + * Save URL, title, description (unlimited size). Classify links with tags (with autocomplete) * Tag renaming, merging and deletion. * Automatic thumbnails for various services (imgur, imageshack.us, flickr, youtube, vimeo, dailymotion…) * Automatic conversion of URLs to clickable links in descriptions. Support for http/ftp/file/apt protocols. * Save links as public or private - * 1-clic access to your private links/notes + * 1-click access to your private links/notes * Browse links by page, filter by tag or use the full text search engine * Permalinks (with QR-Code) for easy reference * Tag cloud diff --git a/inc/shaarli.css b/inc/shaarli.css index 508398483..a28e753dc 100644 --- a/inc/shaarli.css +++ b/inc/shaarli.css @@ -1,4 +1,4 @@ -/* CSS Stylsheet for Shaarli - http://sebsauvage.net/wiki/doku.php?id=php:shaarli */ +/* Cascading Style Sheet for Shaarli - http://sebsauvage.net/wiki/doku.php?id=php:shaarli */ /* CSS Reset from Yahoo to cope with browsers CSS inconsistencies. */ /* @@ -292,7 +292,7 @@ font-size:9pt; .picwall_pictureframe img { max-width: 100%;height: auto; } /* Adapt the width of the image */ .picwall_pictureframe a {text-decoration:none;} -/* CSS to show title when hovering an image - no javascript required. */ +/* CSS to show title when hovering an image - no JavaScript required. */ .picwall_pictureframe span.info {display: none;} .picwall_pictureframe:hover span.info { display:block; @@ -407,13 +407,13 @@ div.dailyEntryDescription overflow:auto; } -/* Common css screwdriver */ +/* Common CSS screwdriver */ .clear{ clear:both; } /* For lazy images loading in picture wall. - using http://www.appelsiini.net/projects/lazyload + Using http://www.appelsiini.net/projects/lazyload */ .lazyimage { display:none; } @@ -465,4 +465,4 @@ div.dailyEntryDescription { font-size:10pt; } } /* Highlight search results */ -.highlight { background-color: #FFFF33; } \ No newline at end of file +.highlight { background-color: #FFFF33; } diff --git a/index.php b/index.php index 1d13e55c2..9a48e0e80 100644 --- a/index.php +++ b/index.php @@ -1,9 +1,9 @@ '); // Suffix to encapsulate data in php code. +define('PHPPREFIX',''); // Suffix to encapsulate data in PHP code. // Force cookie path (but do not change lifetime) $cookie=session_get_cookie_params(); @@ -46,8 +46,8 @@ // Set session parameters on server side. define('INACTIVITY_TIMEOUT',3600); // (in seconds). If the user does not access any page within this time, his/her session is considered expired. ini_set('session.use_cookies', 1); // Use cookies to store session. -ini_set('session.use_only_cookies', 1); // Force cookies for session (phpsessionID forbidden in URL) -ini_set('session.use_trans_sid', false); // Prevent php to use sessionID in URL if cookies are disabled. +ini_set('session.use_only_cookies', 1); // Force cookies for session (phpsessionID forbidden in URL). +ini_set('session.use_trans_sid', false); // Prevent PHP to use sessionID in URL if cookies are disabled. session_name('shaarli'); if (session_id() == '') session_start(); // Start session if needed (Some server auto-start sessions). @@ -83,7 +83,7 @@ function stripslashes_deep($value) { $value = is_array($value) ? array_map('stri header("Cache-Control: post-check=0, pre-check=0", false); header("Pragma: no-cache"); -// Directories creations (Note that your web host may require differents rights than 705.) +// Directories creations (Note that your web host may require different rights than 705.) if (!is_writable(realpath(dirname(__FILE__)))) die('
ERROR: Shaarli does not have the right to write in its own directory ('.realpath(dirname(__FILE__)).').
'); if (!is_dir($GLOBALS['config']['DATADIR'])) { mkdir($GLOBALS['config']['DATADIR'],0705); chmod($GLOBALS['config']['DATADIR'],0705); } if (!is_dir('tmp')) { mkdir('tmp',0705); chmod('tmp',0705); } // For RainTPL temporary files. @@ -114,13 +114,13 @@ function stripslashes_deep($value) { $value = is_array($value) ? array_map('stri autoLocale(); // Sniff browser language and set date format accordingly. header('Content-Type: text/html; charset=utf-8'); // We use UTF-8 for proper international characters handling. -// Check php version +// Check PHP version. function checkphpversion() { if (version_compare(PHP_VERSION, '5.1.0') < 0) { header('Content-Type: text/plain; charset=utf-8'); - echo 'Your server supports php '.PHP_VERSION.'. Shaarli requires at least php 5.1.0, and thus cannot run. Sorry.'; + echo 'Your server supports PHP '.PHP_VERSION.'. Shaarli requires at least PHP 5.1.0, and thus cannot run. Sorry.'; exit; } } @@ -139,7 +139,7 @@ function checkUpdate() $version=shaarli_version; list($httpstatus,$headers,$data) = getHTTP('http://sebsauvage.net/files/shaarli_version.txt',2); if (strpos($httpstatus,'200 OK')!==false) $version=$data; - // If failed, nevermind. We don't want to bother the user with that. + // If failed, never mind. We don't want to bother the user with that. file_put_contents($GLOBALS['config']['UPDATECHECK_FILENAME'],$version); // touch file date } // Compare versions: @@ -155,11 +155,11 @@ function checkUpdate() class pageCache { private $url; // Full URL of the page to cache (typically the value returned by pageUrl()) - private $shouldBeCached; // boolean: Should this url be cached ? - private $filename; // Name of the cache file for this url + private $shouldBeCached; // boolean: Should this URL be cached? + private $filename; // Name of the cache file for this URL. /* - $url = url (typically the value returned by pageUrl()) + $url = URL (typically the value returned by pageUrl()) $shouldBeCached = boolean. If false, the cache will be disabled. */ public function __construct($url,$shouldBeCached) @@ -222,7 +222,7 @@ function nl2br_escaped($html) } /* Returns the small hash of a string - eg. smallHash('20111006_131924') --> yZH23w + e.g. smallHash('20111006_131924') --> yZH23w Small hashes: - are unique (well, as unique as crc32, at last) - are always 6 characters long. @@ -239,7 +239,7 @@ function smallHash($text) return $t; } -// In a string, converts urls to clickable links. +// In a string, converts URLs to clickable links. // Function inspired from http://www.php.net/manual/en/function.preg-replace.php#85722 function text2clickable($url) { @@ -260,8 +260,8 @@ function keepMultipleSpaces($text) function autoLocale() { $loc='en_US'; // Default if browser does not send HTTP_ACCEPT_LANGUAGE - if (isset($_SERVER['HTTP_ACCEPT_LANGUAGE'])) // eg. "fr,fr-fr;q=0.8,en;q=0.5,en-us;q=0.3" - { // (It's a bit crude, but it works very well. Prefered language is always presented first.) + if (isset($_SERVER['HTTP_ACCEPT_LANGUAGE'])) // e.g. "fr,fr-fr;q=0.8,en;q=0.5,en-us;q=0.3" + { // (It's a bit crude, but it works very well. Preferred language is always presented first.) if (preg_match('/([a-z]{2}(-[a-z]{2})?)/i',$_SERVER['HTTP_ACCEPT_LANGUAGE'],$matches)) $loc=$matches[1]; } setlocale(LC_TIME,$loc); // LC_TIME = Set local for date/time format only. @@ -303,7 +303,7 @@ function check_auth($login,$password) $hash = sha1($password.$login.$GLOBALS['salt']); if ($login==$GLOBALS['login'] && $hash==$GLOBALS['hash']) { // Login/password is correct. - $_SESSION['uid'] = sha1(uniqid('',true).'_'.mt_rand()); // generate unique random number (different than phpsessionid) + $_SESSION['uid'] = sha1(uniqid('',true).'_'.mt_rand()); // Generate unique random number (different than phpsessionid). $_SESSION['ip']=allIPs(); // We store IP address(es) of the client to make sure session is not hijacked. $_SESSION['username']=$login; $_SESSION['expires_on']=time()+INACTIVITY_TIMEOUT; // Set session expiration. @@ -391,7 +391,7 @@ function ban_canLogin() { if (!ban_canLogin()) die('I said: NO. You are banned for the moment. Go away.'); if (isset($_POST['password']) && tokenOk($_POST['token']) && (check_auth($_POST['login'], $_POST['password']))) - { // Login/password is ok. + { // Login/password is OK. ban_loginOk(); // If user wants to keep the session cookie even after the browser closes: if (!empty($_POST['longlastingsession'])) @@ -401,7 +401,7 @@ function ban_canLogin() $cookiedir = ''; if(dirname($_SERVER['SCRIPT_NAME'])!='/') $cookiedir=dirname($_SERVER["SCRIPT_NAME"]).'/'; session_set_cookie_params($_SESSION['longlastingsession'],$cookiedir,$_SERVER['SERVER_NAME']); // Set session cookie expiration on client side - // Note: Never forget the trailing slash on the cookie path ! + // Note: Never forget the trailing slash on the cookie path! session_regenerate_id(true); // Send cookie with new expiration date to browser. } else // Standard session expiration (=when browser closes) @@ -433,7 +433,7 @@ function ban_canLogin() // Misc utility functions: // Returns the server URL (including port and http/https), without path. -// eg. "http://myserver.com:8080" +// e.g. "http://myserver.com:8080" // You can append $_SERVER['SCRIPT_NAME'] to get the current script URL. function serverUrl() { @@ -443,24 +443,24 @@ function serverUrl() } // Returns the absolute URL of current script, without the query. -// (eg. http://sebsauvage.net/links/) +// (e.g. http://sebsauvage.net/links/) function indexUrl() { $scriptname = $_SERVER["SCRIPT_NAME"]; // If the script is named 'index.php', we remove it (for better looking URLs, - // eg. http://mysite.com/shaarli/?abcde instead of http://mysite.com/shaarli/index.php?abcde) + // e.g. http://mysite.com/shaarli/?abcde instead of http://mysite.com/shaarli/index.php?abcde) if (endswith($scriptname,'index.php')) $scriptname = substr($scriptname,0,strlen($scriptname)-9); return serverUrl() . $scriptname; } // Returns the absolute URL of current script, WITH the query. -// (eg. http://sebsauvage.net/links/?toto=titi&spamspamspam=humbug) +// (e.g. http://sebsauvage.net/links/?toto=titi&spamspamspam=humbug) function pageUrl() { return indexUrl().(!empty($_SERVER["QUERY_STRING"]) ? '?'.$_SERVER["QUERY_STRING"] : ''); } -// Convert post_max_size/upload_max_filesize (eg.'16M') parameters to bytes. +// Convert post_max_size/upload_max_filesize (e.g.'16M') parameters to bytes. function return_bytes($val) { $val = trim($val); $last=strtolower($val[strlen($val)-1]); @@ -481,7 +481,7 @@ function getMaxFileSize() $size2 = return_bytes(ini_get('upload_max_filesize')); // Return the smaller of two: $maxsize = min($size1,$size2); - // FIXME: Then convert back to readable notations ? (eg. 2M instead of 2000000) + // FIXME: Then convert back to readable notations? (e.g. 2M instead of 2000000) return $maxsize; } @@ -529,7 +529,7 @@ function linkdate2iso8601($linkdate) function linkdate2locale($linkdate) { return utf8_encode(strftime('%c',linkdate2timestamp($linkdate))); // %c is for automatic date format according to locale. - // Note that if you use a local which is not installed on your webserver, + // Note that if you use a locale which is not installed on your webserver, // the date will not be displayed in the chosen locale, but probably in US notation. } @@ -551,10 +551,10 @@ function http_parse_headers_shaarli( $headers ) } /* GET an URL. - Input: $url : url to get (http://...) - $timeout : Network timeout (will wait this many seconds for an anwser before giving up). - Output: An array. [0] = HTTP status message (eg. "HTTP/1.1 200 OK") or error message - [1] = associative array containing HTTP response headers (eg. echo getHTTP($url)[1]['Content-Type']) + Input: $url : URL to get (http://...) + $timeout : Network timeout (will wait this many seconds for an answer before giving up). + Output: An array. [0] = HTTP status message (e.g. "HTTP/1.1 200 OK") or error message + [1] = associative array containing HTTP response headers (e.g. echo getHTTP($url)[1]['Content-Type']) [2] = data Example: list($httpstatus,$headers,$data) = getHTTP('http://sebauvage.net/'); if (strpos($httpstatus,'200 OK')!==false) @@ -570,11 +570,11 @@ function getHTTP($url,$timeout=30) $context = stream_context_create($options); $data=file_get_contents($url,false,$context,-1, 4000000); // We download at most 4 Mb from source. if (!$data) { return array('HTTP Error',array(),''); } - $httpStatus=$http_response_header[0]; // eg. "HTTP/1.1 200 OK" + $httpStatus=$http_response_header[0]; // e.g. "HTTP/1.1 200 OK" $responseHeaders=http_parse_headers_shaarli($http_response_header); return array($httpStatus,$responseHeaders,$data); } - catch (Exception $e) // getHTTP *can* fail silentely (we don't care if the title cannot be fetched) + catch (Exception $e) // getHTTP *can* fail silently (we don't care if the title cannot be fetched) { return array($e->getMessage(),'',''); } @@ -600,14 +600,14 @@ function getToken() return $rnd; } -// Tells if a token is ok. Using this function will destroy the token. -// true=token is ok. +// Tells if a token is OK. Using this function will destroy the token. +// true=token is OK. function tokenOk($token) { if (isset($_SESSION['tokens'][$token])) { unset($_SESSION['tokens'][$token]); // Token is used: destroy it. - return true; // Token is ok. + return true; // Token is OK. } return false; // Wrong token, or already used. } @@ -642,7 +642,7 @@ private function initialize() $this->tpl->assign('version',shaarli_version); $this->tpl->assign('scripturl',indexUrl()); $this->tpl->assign('pagetitle','Shaarli'); - $this->tpl->assign('privateonly',!empty($_SESSION['privateonly'])); // Show only private links ? + $this->tpl->assign('privateonly',!empty($_SESSION['privateonly'])); // Show only private links? if (!empty($GLOBALS['title'])) $this->tpl->assign('pagetitle',$GLOBALS['title']); if (!empty($GLOBALS['pagetitle'])) $this->tpl->assign('pagetitle',$GLOBALS['pagetitle']); $this->tpl->assign('shaarlititle',empty($GLOBALS['title']) ? 'Shaarli': $GLOBALS['title'] ); @@ -657,7 +657,7 @@ public function assign($what,$where) } // Render a specific page (using a template). - // eg. pb.renderPage('picwall') + // e.g. pb.renderPage('picwall') public function renderPage($page) { if ($this->tpl===false) $this->initialize(); // Lazy initialization @@ -672,14 +672,14 @@ public function renderPage($page) $mylinks = new linkdb(); echo $mylinks['20110826_161819']['title']; foreach($mylinks as $link) - echo $link['title'].' at url '.$link['url'].' ; description:'.$link['description']; + echo $link['title'].' at URL '.$link['url'].' ; description:'.$link['description']; Available keys: title : Title of the link - url : URL of the link. Can be absolute or relative. Relative URLs are permalinks (eg.'?m-ukcw') + url : URL of the link. Can be absolute or relative. Relative URLs are permalinks (e.g. '?m-ukcw') description : description of the entry - private : Is this link private ? 0=no, other value=yes - linkdate : date of the creation of this entry, in the form YYYYMMDD_HHMMSS (eg.'20110914_192317') + private : Is this link private? 0=no, other value=yes + linkdate : date of the creation of this entry, in the form YYYYMMDD_HHMMSS (e.g. '20110914_192317') tags : tags attached to this entry (separated by spaces) We implement 3 interfaces: @@ -689,15 +689,15 @@ public function renderPage($page) */ class linkdb implements Iterator, Countable, ArrayAccess { - private $links; // List of links (associative array. Key=linkdate (eg. "20110823_124546"), value= associative array (keys:title,description...) + private $links; // List of links (associative array. Key=linkdate (e.g. "20110823_124546"), value= associative array (keys:title,description...) private $urls; // List of all recorded URLs (key=url, value=linkdate) for fast reserve search (url-->linkdate) private $keys; // List of linkdate keys (for the Iterator interface implementation) private $position; // Position in the $this->keys array. (for the Iterator interface implementation.) - private $loggedin; // Is the used logged in ? (used to filter private links) + private $loggedin; // Is the used logged in? (used to filter private links) // Constructor: function __construct($isLoggedIn) - // Input : $isLoggedIn : is the used logged in ? + // Input : $isLoggedIn : is the used logged in? { $this->loggedin = $isLoggedIn; $this->checkdb(); // Make sure data file exists. @@ -711,7 +711,7 @@ public function count() { return count($this->links); } public function offsetSet($offset, $value) { if (!$this->loggedin) die('You are not authorized to add a link.'); - if (empty($value['linkdate']) || empty($value['url'])) die('Internal Error: A link should always have a linkdate and url.'); + if (empty($value['linkdate']) || empty($value['url'])) die('Internal Error: A link should always have a linkdate and URL.'); if (empty($offset)) die('You must specify a key.'); $this->links[$offset] = $value; $this->urls[$value['url']]=$offset; @@ -733,12 +733,12 @@ function next() { ++$this->position; } // go to next item function valid() { return isset($this->keys[$this->position]); } // Check if current position is valid. // ---- Misc methods - private function checkdb() // Check if db directory and file exists. + private function checkdb() // Check if DB directory and file exists. { if (!file_exists($GLOBALS['config']['DATASTORE'])) // Create a dummy database for example. { $this->links = array(); - $link = array('title'=>'Shaarli - sebsauvage.net','url'=>'http://sebsauvage.net/wiki/doku.php?id=php:shaarli','description'=>'Welcome to Shaarli ! This is a bookmark. To edit or delete me, you must first login.','private'=>0,'linkdate'=>'20110914_190000','tags'=>'opensource software'); + $link = array('title'=>'Shaarli - sebsauvage.net','url'=>'http://sebsauvage.net/wiki/doku.php?id=php:shaarli','description'=>'Welcome to Shaarli! This is a bookmark. To edit or delete me, you must first login.','private'=>0,'linkdate'=>'20110914_190000','tags'=>'opensource software'); $this->links[$link['linkdate']] = $link; $link = array('title'=>'My secret stuff... - Pastebin.com','url'=>'http://pastebin.com/smCEEeSn','description'=>'SShhhh!! I\'m a private link only YOU can see. You can delete me too.','private'=>1,'linkdate'=>'20110914_074522','tags'=>'secretstuff'); $this->links[$link['linkdate']] = $link; @@ -774,19 +774,19 @@ public function savedb() invalidateCaches(); } - // Returns the link for a given URL (if it exists). false it does not exist. + // Returns the link for a given URL (if it exists). False if it does not exist. public function getLinkFromUrl($url) { if (isset($this->urls[$url])) return $this->links[$this->urls[$url]]; return false; } - // Case insentitive search among links (in url, title and description). Returns filtered list of links. - // eg. print_r($mydb->filterFulltext('hollandais')); + // Case insensitive search among links (in the URL, title and description). Returns filtered list of links. + // e.g. print_r($mydb->filterFulltext('hollandais')); public function filterFulltext($searchterms) { // FIXME: explode(' ',$searchterms) and perform a AND search. - // FIXME: accept double-quotes to search for a string "as is" ? + // FIXME: accept double-quotes to search for a string "as is"? $filtered=array(); $s = strtolower($searchterms); foreach($this->links as $l) @@ -803,7 +803,7 @@ public function filterFulltext($searchterms) // Filter by tag. // You can specify one or more tags (tags can be separated by space or comma). - // eg. print_r($mydb->filterTags('linux programming')); + // e.g. print_r($mydb->filterTags('linux programming')); public function filterTags($tags,$casesensitive=false) { $t = str_replace(',',' ',($casesensitive?$tags:strtolower($tags))); @@ -819,9 +819,9 @@ public function filterTags($tags,$casesensitive=false) return $filtered; } - // Filter by day. Day must be in the form 'YYYYMMDD' (eg. '20120125') + // Filter by day. Day must be in the form 'YYYYMMDD' (e.g. '20120125') // Sort order is: older articles first. - // eg. print_r($mydb->filterDay('20120125')); + // e.g. print_r($mydb->filterDay('20120125')); public function filterDay($day) { $filtered=array(); @@ -876,13 +876,13 @@ public function days() } // ------------------------------------------------------------------------------------------ -// Ouput the last 50 links in RSS 2.0 format. +// Output the last 50 links in RSS 2.0 format. function showRSS() { header('Content-Type: application/rss+xml; charset=utf-8'); // $usepermalink : If true, use permalink instead of final link. - // User just has to add 'permalink' in URL parameters. eg. http://mysite.com/shaarli/?do=rss&permalinks + // User just has to add 'permalink' in URL parameters. e.g. http://mysite.com/shaarli/?do=rss&permalinks $usepermalinks = isset($_GET['permalinks']); // Cache system @@ -891,9 +891,9 @@ function showRSS() $cached = $cache->cachedVersion(); if (!empty($cached)) { echo $cached; exit; } // If cached was not found (or not usable), then read the database and build the response: - $LINKSDB=new linkdb(isLoggedIn() || $GLOBALS['config']['OPEN_SHAARLI']); // Read links from database (and filter private links if used it not logged in). + $LINKSDB=new linkdb(isLoggedIn() || $GLOBALS['config']['OPEN_SHAARLI']); // Read links from database (and filter private links if user is not logged in). - // Optionnaly filter the results: + // Optionally filter the results: $linksToDisplay=array(); if (!empty($_GET['searchterm'])) $linksToDisplay = $LINKSDB->filterFulltext($_GET['searchterm']); elseif (!empty($_GET['searchtags'])) $linksToDisplay = $LINKSDB->filterTags(trim($_GET['searchtags'])); @@ -945,13 +945,13 @@ function showRSS() } // ------------------------------------------------------------------------------------------ -// Ouput the last 50 links in ATOM format. +// Output the last 50 links in ATOM format. function showATOM() { header('Content-Type: application/atom+xml; charset=utf-8'); // $usepermalink : If true, use permalink instead of final link. - // User just has to add 'permalink' in URL parameters. eg. http://mysite.com/shaarli/?do=atom&permalinks + // User just has to add 'permalink' in URL parameters. e.g. http://mysite.com/shaarli/?do=atom&permalinks $usepermalinks = isset($_GET['permalinks']); // Cache system @@ -963,7 +963,7 @@ function showATOM() $LINKSDB=new linkdb(isLoggedIn() || $GLOBALS['config']['OPEN_SHAARLI']); // Read links from database (and filter private links if used it not logged in). - // Optionnaly filter the results: + // Optionally filter the results: $linksToDisplay=array(); if (!empty($_GET['searchterm'])) $linksToDisplay = $LINKSDB->filterFulltext($_GET['searchterm']); elseif (!empty($_GET['searchtags'])) $linksToDisplay = $LINKSDB->filterTags(trim($_GET['searchtags'])); @@ -1054,7 +1054,7 @@ function showDailyRSS() if (empty($days[$day])) $days[$day]=array(); $days[$day][]=$linkdate; } - if (count($days)>$nb_of_days) break; // Have we collected enough days ? + if (count($days)>$nb_of_days) break; // Have we collected enough days? } // Build the RSS feed. @@ -1133,7 +1133,7 @@ function showDaily() } /* We need to spread the articles on 3 columns. - I did not want to use a javascript lib like http://masonry.desandro.com/ + I did not want to use a JavaScript lib like http://masonry.desandro.com/ so I manually spread entries with a simple method: I roughly evaluate the height of a div according to title and description length. */ @@ -1144,7 +1144,7 @@ function showDaily() // Roughly estimate length of entry (by counting characters) // Title: 30 chars = 1 line. 1 line is 30 pixels height. // Description: 836 characters gives roughly 342 pixel height. - // This is not perfect, but it's usually ok. + // This is not perfect, but it's usually OK. $length=strlen($link['title'])+(342*strlen($link['description']))/836; if ($link['thumbnail']) $length +=100; // 1 thumbnails roughly takes 100 pixels height. // Then put in column which is the less filled: @@ -1197,7 +1197,7 @@ function renderPage() // -------- Picture wall if (isset($_SERVER["QUERY_STRING"]) && startswith($_SERVER["QUERY_STRING"],'do=picwall')) { - // Optionnaly filter the results: + // Optionally filter the results: $links=array(); if (!empty($_GET['searchterm'])) $links = $LINKSDB->filterFulltext($_GET['searchterm']); elseif (!empty($_GET['searchtags'])) $links = $LINKSDB->filterTags(trim($_GET['searchtags'])); @@ -1277,7 +1277,7 @@ function renderPage() if (isset($_GET['linksperpage'])) { if (is_numeric($_GET['linksperpage'])) { $_SESSION['LINKS_PER_PAGE']=abs(intval($_GET['linksperpage'])); } - // Make sure the referer is from Shaarli itself. + // Make sure the referrer is Shaarli itself. $referer = '?'; if (!empty($_SERVER['HTTP_REFERER']) && strcmp(parse_url($_SERVER['HTTP_REFERER'],PHP_URL_HOST),$_SERVER['SERVER_NAME'])==0) $referer = $_SERVER['HTTP_REFERER']; @@ -1296,7 +1296,7 @@ function renderPage() { unset($_SESSION['privateonly']); // See all links } - // Make sure the referer is from Shaarli itself. + // Make sure the referrer is Shaarli itself. $referer = '?'; if (!empty($_SERVER['HTTP_REFERER']) && strcmp(parse_url($_SERVER['HTTP_REFERER'],PHP_URL_HOST),$_SERVER['SERVER_NAME'])==0) $referer = $_SERVER['HTTP_REFERER']; @@ -1307,7 +1307,7 @@ function renderPage() // -------- Handle other actions allowed for non-logged in users: if (!isLoggedIn()) { - // User tries to post new link but is not loggedin: + // User tries to post new link but is not logged in: // Show login screen, then redirect to ?post=... if (isset($_GET['post'])) { @@ -1317,7 +1317,7 @@ function renderPage() $PAGE = new pageBuilder; buildLinkList($PAGE,$LINKSDB); // Compute list of links to display $PAGE->renderPage('linklist'); - exit; // Never remove this one ! All operations below are reserved for logged in user. + exit; // Never remove this one! All operations below are reserved for logged in user. } // -------- All other functions are reserved for the registered user: @@ -1338,7 +1338,7 @@ function renderPage() if ($GLOBALS['config']['OPEN_SHAARLI']) die('You are not supposed to change a password on an Open Shaarli.'); if (!empty($_POST['setpassword']) && !empty($_POST['oldpassword'])) { - if (!tokenOk($_POST['token'])) die('Wrong token.'); // Go away ! + if (!tokenOk($_POST['token'])) die('Wrong token.'); // Go away! // Make sure old password is correct. $oldhash = sha1($_POST['oldpassword'].$GLOBALS['login'].$GLOBALS['salt']); @@ -1365,7 +1365,7 @@ function renderPage() { if (!empty($_POST['title']) ) { - if (!tokenOk($_POST['token'])) die('Wrong token.'); // Go away ! + if (!tokenOk($_POST['token'])) die('Wrong token.'); // Go away! $tz = 'UTC'; if (!empty($_POST['continent']) && !empty($_POST['city'])) if (isTZvalid($_POST['continent'],$_POST['city'])) @@ -1388,7 +1388,7 @@ function renderPage() $PAGE->assign('title',htmlspecialchars( empty($GLOBALS['title']) ? '' : $GLOBALS['title'] , ENT_QUOTES)); $PAGE->assign('redirector',htmlspecialchars( empty($GLOBALS['redirector']) ? '' : $GLOBALS['redirector'] , ENT_QUOTES)); list($timezone_form,$timezone_js) = templateTZform($GLOBALS['timezone']); - $PAGE->assign('timezone_form',$timezone_form); // FIXME: put entire tz form generation in template ? + $PAGE->assign('timezone_form',$timezone_form); // FIXME: Put entire tz form generation in template? $PAGE->assign('timezone_js',$timezone_js); $PAGE->renderPage('configure'); exit; @@ -1412,7 +1412,7 @@ function renderPage() if (!empty($_POST['deletetag']) && !empty($_POST['fromtag'])) { $needle=trim($_POST['fromtag']); - $linksToAlter = $LINKSDB->filterTags($needle,true); // true for case-sensitive tag search. + $linksToAlter = $LINKSDB->filterTags($needle,true); // True for case-sensitive tag search. foreach($linksToAlter as $key=>$value) { $tags = explode(' ',trim($value['tags'])); @@ -1420,7 +1420,7 @@ function renderPage() $value['tags']=trim(implode(' ',$tags)); $LINKSDB[$key]=$value; } - $LINKSDB->savedb(); // save to disk + $LINKSDB->savedb(); // Save to disk. echo ''; exit; } @@ -1429,21 +1429,21 @@ function renderPage() if (!empty($_POST['renametag']) && !empty($_POST['fromtag']) && !empty($_POST['totag'])) { $needle=trim($_POST['fromtag']); - $linksToAlter = $LINKSDB->filterTags($needle,true); // true for case-sensitive tag search. + $linksToAlter = $LINKSDB->filterTags($needle,true); // True for case-sensitive tag search. foreach($linksToAlter as $key=>$value) { $tags = explode(' ',trim($value['tags'])); - $tags[array_search($needle,$tags)] = trim($_POST['totag']); // Remplace tags value. + $tags[array_search($needle,$tags)] = trim($_POST['totag']); // Replace tags value. $value['tags']=trim(implode(' ',$tags)); $LINKSDB[$key]=$value; } - $LINKSDB->savedb(); // save to disk + $LINKSDB->savedb(); // Save to disk. echo ''; exit; } } - // -------- User wants to add a link without using the bookmarklet: show form. + // -------- User wants to add a link without using the bookmarklet: Show form. if (isset($_SERVER["QUERY_STRING"]) && startswith($_SERVER["QUERY_STRING"],'do=addlink')) { $PAGE = new pageBuilder; @@ -1455,7 +1455,7 @@ function renderPage() // -------- User clicked the "Save" button when editing a link: Save link to database. if (isset($_POST['save_edit'])) { - if (!tokenOk($_POST['token'])) die('Wrong token.'); // Go away ! + if (!tokenOk($_POST['token'])) die('Wrong token.'); // Go away! $tags = trim(preg_replace('/\s\s+/',' ', $_POST['lf_tags'])); // Remove multiple spaces. $linkdate=$_POST['lf_linkdate']; $url = trim($_POST['lf_url']); @@ -1465,7 +1465,7 @@ function renderPage() 'linkdate'=>$linkdate,'tags'=>str_replace(',',' ',$tags)); if ($link['title']=='') $link['title']=$link['url']; // If title is empty, use the URL as title. $LINKSDB[$linkdate] = $link; - $LINKSDB->savedb(); // save to disk + $LINKSDB->savedb(); // Save to disk. pubsubhub(); // If we are called from the bookmarklet, we must close the popup: @@ -1479,7 +1479,7 @@ function renderPage() // -------- User clicked the "Cancel" button when editing a link. if (isset($_POST['cancel_edit'])) { - // If we are called from the bookmarklet, we must close the popup; + // If we are called from the bookmarklet, we must close the popup: if (isset($_GET['source']) && $_GET['source']=='bookmarklet') { echo ''; exit; } $returnurl = ( isset($_POST['returnurl']) ? $_POST['returnurl'] : '?' ); $returnurl .= '#'.smallHash($_POST['lf_linkdate']); // Scroll to the link which has been edited. @@ -1487,12 +1487,12 @@ function renderPage() exit; } - // -------- User clicked the "Delete" button when editing a link : Delete link from database. + // -------- User clicked the "Delete" button when editing a link: Delete link from database. if (isset($_POST['delete_link'])) { if (!tokenOk($_POST['token'])) die('Wrong token.'); // We do not need to ask for confirmation: - // - confirmation is handled by javascript + // - confirmation is handled by JavaScript // - we are protected from XSRF by the token. $linkdate=$_POST['lf_linkdate']; unset($LINKSDB[$linkdate]); @@ -1540,7 +1540,7 @@ function renderPage() $title = (empty($_GET['title']) ? '' : $_GET['title'] ); // Get title if it was provided in URL (by the bookmarklet). $description=''; $tags=''; $private=0; if (($url!='') && parse_url($url,PHP_URL_SCHEME)=='') $url = 'http://'.$url; - // If this is an HTTP link, we try go get the page to extact the title (otherwise we will to straight to the edit form.) + // If this is an HTTP link, we try go get the page to extract the title (otherwise we will to straight to the edit form.) if (empty($title) && parse_url($url,PHP_URL_SCHEME)=='http') { list($status,$headers,$data) = getHTTP($url,4); // Short timeout to keep the application responsive. @@ -1573,7 +1573,7 @@ function renderPage() exit; } $exportWhat=$_GET['what']; - if (!array_intersect(array('all','public','private'),array($exportWhat))) die('What are you trying to export ???'); + if (!array_intersect(array('all','public','private'),array($exportWhat))) die('What are you trying to export???'); header('Content-Type: text/html; charset=utf-8'); header('Content-disposition: attachment; filename=bookmarks_'.$exportWhat.'_'.strval(date('Ymd_His')).'.html'); @@ -1646,8 +1646,8 @@ function importFile() $filename=$_FILES['filetoupload']['name']; $filesize=$_FILES['filetoupload']['size']; $data=file_get_contents($_FILES['filetoupload']['tmp_name']); - $private = (empty($_POST['private']) ? 0 : 1); // Should the links be imported as private ? - $overwrite = !empty($_POST['overwrite']) ; // Should the imported links overwrite existing ones ? + $private = (empty($_POST['private']) ? 0 : 1); // Should the links be imported as private? + $overwrite = !empty($_POST['overwrite']) ; // Should the imported links overwrite existing ones? $import_count=0; // Sniff file type: @@ -1658,7 +1658,7 @@ function importFile() if ($type=='netscape') { // This is a standard Netscape-style bookmark file. - // This format is supported by all browsers (except IE, of course), also delicious, diigo and others. + // This format is supported by all browsers (except IE, of course), also Delicious, Diigo and others. foreach(explode('
',$data) as $html) // explode is very fast { $link = array('linkdate'=>'','title'=>'','url'=>'','description'=>'','tags'=>'','private'=>0); @@ -1688,14 +1688,14 @@ function importFile() // Make sure date/time is not already used by another link. // (Some bookmark files have several different links with the same ADD_DATE) - // We increment date by 1 second until we find a date which is not used in db. + // We increment date by 1 second until we find a date which is not used in DB. // (so that links that have the same date/time are more or less kept grouped by date, but do not conflict.) while (!empty($LINKSDB[date('Ymd_His',$raw_add_date)])) { $raw_add_date++; }// Yes, I know it's ugly. $link['linkdate']=date('Ymd_His',$raw_add_date); $LINKSDB[$link['linkdate']] = $link; $import_count++; } - else // link already present in database. + else // Link already present in database. { if ($overwrite) { // If overwrite is required, we import link data, except date/time. @@ -1746,13 +1746,13 @@ function buildLinkList($PAGE,$LINKSDB) { header($_SERVER["SERVER_PROTOCOL"]." 404 Not Found"); echo '

404 Not found.

Oh crap. The link you are trying to reach does not exist or has been deleted.'; - echo '
You would mind clicking here ?'; + echo '
You would mind clicking here?'; exit; } $search_type='permalink'; } else - $linksToDisplay = $LINKSDB; // otherwise, display without filtering. + $linksToDisplay = $LINKSDB; // Otherwise, display without filtering. // Option: Show only private links if (!empty($_SESSION['privateonly'])) @@ -1766,11 +1766,11 @@ function buildLinkList($PAGE,$LINKSDB) } // ---- Handle paging. - /* Can someone explain to me why you get the following error when using array_keys() on an object which implements the interface ArrayAccess ??? + /* Can someone explain to me why you get the following error when using array_keys() on an object which implements the interface ArrayAccess??? "Warning: array_keys() expects parameter 1 to be array, object given in ... " - If my class implements ArrayAccess, why won't array_keys() accept it ? ( $keys=array_keys($linksToDisplay); ) + If my class implements ArrayAccess, why won't array_keys() accept it? ( $keys=array_keys($linksToDisplay); ) */ - $keys=array(); foreach($linksToDisplay as $key=>$value) { $keys[]=$key; } // Stupid and ugly. Thanks php. + $keys=array(); foreach($linksToDisplay as $key=>$value) { $keys[]=$key; } // Stupid and ugly. Thanks PHP. // If there is only a single link, we change on-the-fly the title of the page. if (count($linksToDisplay)==1) $GLOBALS['pagetitle'] = $linksToDisplay[$keys[0]]['title'].' - '.$GLOBALS['title']; @@ -1817,7 +1817,7 @@ function buildLinkList($PAGE,$LINKSDB) $PAGE->assign('result_count',count($linksToDisplay)); $PAGE->assign('search_type',$search_type); $PAGE->assign('search_crits',$search_crits); - $PAGE->assign('redirector',empty($GLOBALS['redirector']) ? '' : $GLOBALS['redirector']); // optional redirector URL + $PAGE->assign('redirector',empty($GLOBALS['redirector']) ? '' : $GLOBALS['redirector']); // Optional redirector URL. $PAGE->assign('token',$token); $PAGE->assign('links',$linkDisp); return; @@ -1825,9 +1825,9 @@ function buildLinkList($PAGE,$LINKSDB) // Compute the thumbnail for a link. // -// with a link to the original URL. +// With a link to the original URL. // Understands various services (youtube.com...) -// Input: $url = url for which the thumbnail must be found. +// Input: $url = URL for which the thumbnail must be found. // $href = if provided, this URL will be followed instead of $url // Returns an associative array with thumbnail attributes (src,href,width,height,style,alt) // Some of them may be missing. @@ -1838,8 +1838,8 @@ function computeThumbnail($url,$href=false) if ($href==false) $href=$url; // For most hosts, the URL of the thumbnail can be easily deduced from the URL of the link. - // (eg. http://www.youtube.com/watch?v=spVypYk4kto ---> http://img.youtube.com/vi/spVypYk4kto/default.jpg ) - // ^^^^^^^^^^^ ^^^^^^^^^^^ + // (e.g. http://www.youtube.com/watch?v=spVypYk4kto ---> http://img.youtube.com/vi/spVypYk4kto/default.jpg ) + // ^^^^^^^^^^^ ^^^^^^^^^^^ $domain = parse_url($url,PHP_URL_HOST); if ($domain=='youtube.com' || $domain=='www.youtube.com') { @@ -1911,17 +1911,17 @@ function computeThumbnail($url,$href=false) ) { if ($domain=='vimeo.com') - { // Make sure this vimeo url points to a video (/xxx... where xxx is numeric) + { // Make sure this vimeo URL points to a video (/xxx... where xxx is numeric) $path = parse_url($url,PHP_URL_PATH); if (!preg_match('!/\d+.+?!',$path)) return array(); // This is not a single video URL. } if ($domain=='xkcd.com' || endsWith($domain,'.xkcd.com')) - { // Make sure this url points to a single comic (/xxx... where xxx is numeric) + { // Make sure this URL points to a single comic (/xxx... where xxx is numeric) $path = parse_url($url,PHP_URL_PATH); if (!preg_match('!/\d+.+?!',$path)) return array(); } if ($domain=='ted.com' || endsWith($domain,'.ted.com')) - { // Make sure this TED url points to a video (/talks/...) + { // Make sure this TED URL points to a video (/talks/...) $path = parse_url($url,PHP_URL_PATH); if ("/talks/" !== substr($path,0,7)) return array(); // This is not a single video URL. } @@ -1948,7 +1948,7 @@ function computeThumbnail($url,$href=false) // Returns the HTML code to display a thumbnail for a link // with a link to the original URL. // Understands various services (youtube.com...) -// Input: $url = url for which the thumbnail must be found. +// Input: $url = URL for which the thumbnail must be found. // $href = if provided, this URL will be followed instead of $url // Returns '' if no thumbnail available. function thumbnail($url,$href=false) @@ -1969,7 +1969,7 @@ function thumbnail($url,$href=false) // Returns the HTML code to display a thumbnail for a link // for the picture wall (using lazy image loading) // Understands various services (youtube.com...) -// Input: $url = url for which the thumbnail must be found. +// Input: $url = URL for which the thumbnail must be found. // $href = if provided, this URL will be followed instead of $url // Returns '' if no thumbnail available. function lazyThumbnail($url,$href=false) @@ -1979,7 +1979,7 @@ function lazyThumbnail($url,$href=false) $html=''; - // Lazy image (only loaded by javascript when in the viewport). + // Lazy image (only loaded by JavaScript when in the viewport). if (!empty($GLOBALS['disablejquery'])) // (except if jQuery is disabled) $html.='alert("Shaarli is now configured. Please enter your login/password and start shaaring your links !");document.location=\'?do=login\';'; + echo ''; exit; } @@ -2061,14 +2061,14 @@ function install() exit; } -// Generates the timezone selection form and javascript. +// Generates the timezone selection form and JavaScript. // Input: (optional) current timezone (can be 'UTC/UTC'). It will be pre-selected. // Output: array(html,js) // Example: list($htmlform,$js) = templateTZform('Europe/Paris'); // Europe/Paris pre-selected. -// Returns array('','') if server does not support timezones list. (eg. php 5.1 on free.fr) +// Returns array('','') if server does not support timezones list. (e.g. PHP 5.1 on free.fr) function templateTZform($ptz=false) { - if (function_exists('timezone_identifiers_list')) // because of old php version (5.1) which can be found on free.fr + if (function_exists('timezone_identifiers_list')) // Because of old PHP version (5.1) which can be found on free.fr { // Try to split the provided timezone. if ($ptz==false) { $l=timezone_identifiers_list(); $ptz=$l[0]; } @@ -2077,7 +2077,7 @@ function templateTZform($ptz=false) // Display config form: $timezone_form = ''; $timezone_js = ''; - // The list is in the forme "Europe/Paris", "America/Argentina/Buenos_Aires"... + // The list is in the form "Europe/Paris", "America/Argentina/Buenos_Aires"... // We split the list in continents/cities. $continents = array(); $cities = array(); @@ -2115,9 +2115,9 @@ function templateTZform($ptz=false) function isTZvalid($continent,$city) { $tz = $continent.'/'.$city; - if (function_exists('timezone_identifiers_list')) // because of old php version (5.1) which can be found on free.fr + if (function_exists('timezone_identifiers_list')) // Because of old PHP version (5.1) which can be found on free.fr { - if (in_array($tz, timezone_identifiers_list())) // it's a valid timezone ? + if (in_array($tz, timezone_identifiers_list())) // It's a valid timezone? return true; } return false; @@ -2125,7 +2125,7 @@ function isTZvalid($continent,$city) // Webservices (for use with jQuery/jQueryUI) -// eg. index.php?ws=tags&term=minecr +// e.g. index.php?ws=tags&term=minecr function processWS() { if (empty($_GET['ws']) || empty($_GET['term'])) return; @@ -2133,7 +2133,7 @@ function processWS() $LINKSDB=new linkdb(isLoggedIn() || $GLOBALS['config']['OPEN_SHAARLI']); // Read links from database (and filter private links if used it not logged in). header('Content-Type: application/json; charset=utf-8'); - // Search in tags (case insentitive, cumulative search) + // Search in tags (case insensitive, cumulative search) if ($_GET['ws']=='tags') { $tags=explode(' ',str_replace(',',' ',$term)); $last = array_pop($tags); // Get the last term ("a b c d" ==> "a b c", "d") @@ -2149,7 +2149,7 @@ function processWS() exit; } - // Search a single tag (case sentitive, single tag search) + // Search a single tag (case sensitive, single tag search) if ($_GET['ws']=='singletag') { /* To speed up things, we store list of tags in session */ @@ -2165,7 +2165,7 @@ function processWS() // Re-write configuration file according to globals. // Requires some $GLOBALS to be set (login,hash,salt,title). -// If the config file cannot be saved, an error message is dislayed and the user is redirected to "Tools" menu. +// If the config file cannot be saved, an error message is displayed and the user is redirected to "Tools" menu. // (otherwise, the function simply returns.) function writeConfig() { @@ -2184,12 +2184,12 @@ function writeConfig() } } -/* Because some f*cking services like Flickr require an extra HTTP request to get the thumbnail URL, +/* Because some f*cking services like flickr require an extra HTTP request to get the thumbnail URL, I have deported the thumbnail URL code generation here, otherwise this would slow down page generation. - The following function takes the URL a link (eg. a flickr page) and return the proper thumbnail. - This function is called by passing the url: + The following function takes the URL a link (e.g. a flickr page) and return the proper thumbnail. + This function is called by passing the URL: http://mywebsite.com/shaarli/?do=genthumbnail&hmac=[HMAC]&url=[URL] - [URL] is the URL of the link (eg. a flickr page) + [URL] is the URL of the link (e.g. a flickr page) [HMAC] is the signature for the [URL] (so that these URL cannot be forged). The function below will fetch the image from the webservice and store it in the cache. */ @@ -2197,7 +2197,7 @@ function genThumbnail() { // Make sure the parameters in the URL were generated by us. $sign = hash_hmac('sha256', $_GET['url'], $GLOBALS['salt']); - if ($sign!=$_GET['hmac']) die('Naughty boy !'); + if ($sign!=$_GET['hmac']) die('Naughty boy!'); // Let's see if we don't already have the image for this URL in the cache. $thumbname=hash('sha1',$_GET['url']).'.jpg'; @@ -2222,22 +2222,22 @@ function genThumbnail() if ($domain=='flickr.com' || endsWith($domain,'.flickr.com')) { - // Crude replacement to handle new Flickr domain policy (They prefer www. now) + // Crude replacement to handle new flickr domain policy (They prefer www. now) $url = str_replace('http://flickr.com/','http://www.flickr.com/',$url); - // Is this a link to an image, or to a flickr page ? + // Is this a link to an image, or to a flickr page? $imageurl=''; if (endswith(parse_url($url,PHP_URL_PATH),'.jpg')) - { // This is a direct link to an image. eg. http://farm1.staticflickr.com/5/5921913_ac83ed27bd_o.jpg + { // This is a direct link to an image. e.g. http://farm1.staticflickr.com/5/5921913_ac83ed27bd_o.jpg preg_match('!(http://farm\d+\.staticflickr\.com/\d+/\d+_\w+_)\w.jpg!',$url,$matches); if (!empty($matches[1])) $imageurl=$matches[1].'m.jpg'; } - else // this is a flickr page (html) + else // This is a flickr page (html). { list($httpstatus,$headers,$data) = getHTTP($url,20); // Get the flickr html page. if (strpos($httpstatus,'200 OK')!==false) { - // Flickr now nicely provides the URL of the thumbnail in each flickr page. + // flickr now nicely provides the URL of the thumbnail in each flickr page. preg_match('! \ No newline at end of file +?> diff --git a/tpl/configure.html b/tpl/configure.html index 62296cb2f..e3050c00a 100644 --- a/tpl/configure.html +++ b/tpl/configure.html @@ -18,7 +18,7 @@ Security: Features: - + New link: @@ -29,4 +29,4 @@ {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/import.html b/tpl/import.html index 9e581fc96..5fea92d18 100644 --- a/tpl/import.html +++ b/tpl/import.html @@ -5,7 +5,7 @@ {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/daily.html b/tpl/daily.html index 4bc8568be..e16779218 100644 --- a/tpl/daily.html +++ b/tpl/daily.html @@ -7,7 +7,7 @@
All links of one day
in a single page.
{if="$previousday"} <Previous day{else}<Previous day{/if} - - + - {if="$nextday"}Next day>{else}Next day>{/if}

Daily RSS Feed @@ -15,7 +15,7 @@
The Daily Shaarli
——————————— {$day} ———————————
- + {if="$linksToDisplay"}
{loop="col1"} @@ -39,7 +39,7 @@
{$value.formatedDescription}
{/loop} -
+
{loop="col3"} @@ -51,7 +51,7 @@
{$value.formatedDescription}
{/loop} - + {else}
No articles on this day.
{/if} @@ -60,4 +60,4 @@ {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/install.html b/tpl/install.html index 4034ef1bb..e88606083 100644 --- a/tpl/install.html +++ b/tpl/install.html @@ -3,7 +3,7 @@ {include="includes"}{$timezone_js}
-

Shaarli

+

Shaarli

It looks like it's the first time you run Shaarli. Please configure it:
@@ -18,4 +18,4 @@

Shaarli

{include="page.footer"} - \ No newline at end of file + diff --git a/tpl/linklist.html b/tpl/linklist.html index 37ffeb646..0905e0a68 100644 --- a/tpl/linklist.html +++ b/tpl/linklist.html @@ -48,8 +48,8 @@ {else} permalink - {/if} -
- +
- {$value.url|htmlspecialchars}
{if="$value.tags"}
@@ -66,7 +66,7 @@
{include="page.footer"} -{if="empty($GLOBALS['disablejquery'])"} +{if="empty($GLOBALS['disablejquery'])"} {/if} - \ No newline at end of file + diff --git a/tpl/page.header.html b/tpl/page.header.html index 125b365b1..bdef1632c 100644 --- a/tpl/page.header.html +++ b/tpl/page.header.html @@ -3,7 +3,7 @@
Shaare your links...
{if="!empty($linkcount)"}{$linkcount} links{/if}
{$shaarlititle|htmlspecialchars} - + {if="!empty($_GET['source']) && $_GET['source']=='bookmarklet'"} {ignore} When called as a popup from bookmarklet, do not display menu. {/ignore} {else} diff --git a/tpl/picwall.html b/tpl/picwall.html index 631e0866b..28ec66106 100644 --- a/tpl/picwall.html +++ b/tpl/picwall.html @@ -1,7 +1,7 @@ {include="includes"} -{if="empty($GLOBALS['disablejquery'])"} +{if="empty($GLOBALS['disablejquery'])"} {/if} @@ -18,12 +18,12 @@ {include="page.footer"} -{if="empty($GLOBALS['disablejquery'])"} +{if="empty($GLOBALS['disablejquery'])"} -{/if} +{/if} - \ No newline at end of file + diff --git a/tpl/readme.txt b/tpl/readme.txt index b18deaed9..d1002c70c 100644 --- a/tpl/readme.txt +++ b/tpl/readme.txt @@ -17,7 +17,6 @@ If you want to also add something in the page header (in the dark area), do it h - Example: "Add new link" form: ----------------------------------------------------- @@ -27,7 +26,7 @@ Example: "Add new link" form: {include="page.header"}
- +
@@ -37,6 +36,3 @@ Example: "Add new link" form: ----------------------------------------------------- - - - From 66f2801ddac4c05dbd6b0f333fc9a1cea17d87e3 Mon Sep 17 00:00:00 2001 From: Robert Spencer Date: Sat, 20 Jul 2013 19:20:51 +0200 Subject: [PATCH 4/9] IDs should only be used once, changed to class --- inc/shaarli.css | 8 ++++---- tpl/linklist.paging.html | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/inc/shaarli.css b/inc/shaarli.css index b901b11b5..cb87a9af9 100644 --- a/inc/shaarli.css +++ b/inc/shaarli.css @@ -201,8 +201,8 @@ cursor:pointer; .paging a:hover { color:#FFFFC9; } .paging a:active { color:#fff; } #paging_privatelinks { float:left; } -#paging_linksperpage { float:right; padding-right:5px; } -#paging_current { display:inline; color:#fff; padding:0 20 0 20; } +.paging_linksperpage { float:right; padding-right:5px; } +.paging_current { display:inline; color:#fff; padding:0 20 0 20; } #paging_older { margin-right:15px; } #paging_newer { margin-left:15px; } @@ -451,8 +451,8 @@ a {color:#000!important;text-decoration:none!important;} #tagfilter_value { width:70% !important; } div.qrcode { position:relative; float:left; top:-10px; left:0px; } #paging_privatelinks { float;none; } -#paging_linksperpage { float:none; margin-bottom:10px; font-size:smaller;} -#paging_older,#paging_newer,#paging_linksperpage a { border: 1px solid black; padding:3px 5px 3px 5px; background-color:#666; color:#fff; border-radius: 5px 5px 5px 5px;} +.paging_linksperpage { float:none; margin-bottom:10px; font-size:smaller;} +#paging_older,#paging_newer,.paging_linksperpage a { border: 1px solid black; padding:3px 5px 3px 5px; background-color:#666; color:#fff; border-radius: 5px 5px 5px 5px;} .thumbnail { float:none; height:auto; margin: 0px; text-align:center;} #cloudtag { padding:0px; } div.dailyAbout { float:none; position:relative; width:100%; clear:both; padding:0px; top:0px; left:0px; } diff --git a/tpl/linklist.paging.html b/tpl/linklist.paging.html index b0c119d98..2b235cb75 100644 --- a/tpl/linklist.paging.html +++ b/tpl/linklist.paging.html @@ -10,11 +10,11 @@
{/if} -
+
Links per page: 20 50 100
{if="$previous_page_url"} ◄Older {/if} -
page {$page_current} / {$page_max}
+
page {$page_current} / {$page_max}
{if="$next_page_url"} Newer► {/if} -
\ No newline at end of file + From 0a516ebdb5e778031e62e4cfcb8159d68fa08947 Mon Sep 17 00:00:00 2001 From: Robert Spencer Date: Mon, 22 Jul 2013 22:47:32 +0200 Subject: [PATCH 5/9] Filter HTML through HTML tidy and tweak --- tpl/addlink.html | 28 ++++--- tpl/changepassword.html | 29 +++++--- tpl/changetag.html | 38 +++++++--- tpl/configure.html | 65 ++++++++++------ tpl/daily.html | 154 +++++++++++++++++++++++++++----------- tpl/dailyrss.html | 15 ++-- tpl/editlink.html | 71 ++++++++++-------- tpl/export.html | 31 +++++--- tpl/import.html | 38 ++++++---- tpl/includes.html | 21 +++--- tpl/install.html | 47 ++++++++---- tpl/linklist.html | 157 +++++++++++++++++++++++---------------- tpl/linklist.paging.html | 41 +++++----- tpl/loginform.html | 43 ++++++----- tpl/page.footer.html | 54 +++++++------- tpl/page.header.html | 24 +++--- tpl/page.html | 16 ++-- tpl/picwall.html | 43 ++++++----- tpl/picwall2.html | 30 +++++--- tpl/readme.txt | 41 ++++++---- tpl/tagcloud.html | 28 ++++--- tpl/tools.html | 45 +++++++---- 22 files changed, 652 insertions(+), 407 deletions(-) diff --git a/tpl/addlink.html b/tpl/addlink.html index 38b4437ee..5d079388a 100644 --- a/tpl/addlink.html +++ b/tpl/addlink.html @@ -1,16 +1,22 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + + + {include="page.footer"} diff --git a/tpl/changepassword.html b/tpl/changepassword.html index 48cf5fa3b..da03237d5 100644 --- a/tpl/changepassword.html +++ b/tpl/changepassword.html @@ -1,15 +1,22 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + + + {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/changetag.html b/tpl/changetag.html index b0bd0d064..8b2a2abe6 100644 --- a/tpl/changetag.html +++ b/tpl/changetag.html @@ -1,16 +1,30 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + + + {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/configure.html b/tpl/configure.html index e3050c00a..4cc613508 100644 --- a/tpl/configure.html +++ b/tpl/configure.html @@ -1,32 +1,49 @@ - -{include="includes"} - - -{include="page.footer"} + + Redirector
+ (e.g. http://anonym.to/? will mask the HTTP_REFERER) + + + Security: + + + + Features: + + + + New link: + + + + + + + + + + + {include="page.footer"} diff --git a/tpl/daily.html b/tpl/daily.html index e16779218..bbbd7ca92 100644 --- a/tpl/daily.html +++ b/tpl/daily.html @@ -1,63 +1,133 @@ + -{include="includes"} + +{include="includes"} + + - -
+ + +
- All links of one day
in a single page.
- {if="$previousday"} <Previous day{else}<Previous day{/if} - - - {if="$nextday"}Next day>{else}Next day>{/if} -

- Daily RSS Feed + All links of one day
+ in a single page.
+ {if="$previousday"}<Previous day{else}<Previous day{/if} - {if="$nextday"}Next day>{else}Next day>{/if}
+
+ Daily RSS Feed +
+ +
+ The Daily Shaarli
-
The Daily Shaarli
-
——————————— {$day} ———————————
+ +
+ ——————————— {$day} ——————————— +
+
{if="$linksToDisplay"} -
- {loop="col1"} -
-
permalink
- {if="$value.tags"}
{loop="value.taglist"}{$value|htmlspecialchars} - {/loop}
{/if} -
{$value.title|htmlspecialchars}
- {if="$value.thumbnail"}
{$value.thumbnail}
{/if} -
{$value.formatedDescription}
+
+ {loop="col1"} +
+
+ permalink +
+ + {if="$value.tags"} +
+ {loop="value.taglist"}{$value|htmlspecialchars} - {/loop} +
+ {/if} + +
+ {$value.title|htmlspecialchars} +
+ + {if="$value.thumbnail"} +
+ {$value.thumbnail} +
+ {/if} + +
+ {$value.formatedDescription} +
+
+ {/loop} +
+ +
+ {loop="col2"} +
+
+ permalink +
+ + {if="$value.tags"} +
+ {loop="value.taglist"}{$value|htmlspecialchars} - {/loop} +
+ {/if} + +
+ {$value.title|htmlspecialchars} +
+ + {if="$value.thumbnail"} +
+ {$value.thumbnail} +
+ {/if} + +
+ {$value.formatedDescription}
- {/loop} +
+ {/loop} +
+ +
+ {loop="col3"} +
+
+ permalink
-
- {loop="col2"} -
-
permalink
- {if="$value.tags"}
{loop="value.taglist"}{$value|htmlspecialchars} - {/loop}
{/if} -
{$value.title|htmlspecialchars}
- {if="$value.thumbnail"}
{$value.thumbnail}
{/if} -
{$value.formatedDescription}
+ {if="$value.tags"} +
+ {loop="value.taglist"}{$value|htmlspecialchars} - {/loop}
- {/loop} + {/if} + +
+ {$value.title|htmlspecialchars}
-
- {loop="col3"} -
-
permalink
- {if="$value.tags"}
{loop="value.taglist"}{$value|htmlspecialchars} - {/loop}
{/if} -
{$value.title|htmlspecialchars}
- {if="$value.thumbnail"}
{$value.thumbnail}
{/if} -
{$value.formatedDescription}
+ {if="$value.thumbnail"} +
+ {$value.thumbnail}
- {/loop} + {/if} + +
+ {$value.formatedDescription}
+
+ {/loop} +
{else} -
No articles on this day.
+
No articles on this day.
{/if} +
-
-
-
-{include="page.footer"} + +
+ - +
+
+ + {include="page.footer"} diff --git a/tpl/dailyrss.html b/tpl/dailyrss.html index 436e1cd2f..ac9db0a01 100644 --- a/tpl/dailyrss.html +++ b/tpl/dailyrss.html @@ -1,8 +1,9 @@ {loop="links"} -

{$value.title|htmlspecialchars}

- {if="!$GLOBALS['config']['HIDE_TIMESTAMPS']"}{$value.localdate|htmlspecialchars} - {/if}{if="$value.tags"}{$value.tags|htmlspecialchars}{/if}
- {$value.url|htmlspecialchars}
- {if="$value.thumbnail"}{$value.thumbnail}{/if}
- {if="$value.description"}{$value.formatedDescription}{/if} -

-{/loop} \ No newline at end of file +

{$value.title|htmlspecialchars}

+ {if="!$GLOBALS['config']['HIDE_TIMESTAMPS']"}{$value.localdate|htmlspecialchars} - {/if}{if="$value.tags"}{$value.tags|htmlspecialchars}{/if}
+ {$value.url|htmlspecialchars}
+ {if="$value.thumbnail"}{$value.thumbnail}{/if}
+ {if="$value.description"}{$value.formatedDescription}{/if}
+
+
+{/loop} diff --git a/tpl/editlink.html b/tpl/editlink.html index ad549137d..f5337eb31 100644 --- a/tpl/editlink.html +++ b/tpl/editlink.html @@ -1,34 +1,45 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + {if condition="$link.title==''"}onload="document.linkform.lf_title.focus();" + {elseif condition="$link.description==''"}onload="document.linkform.lf_description.focus();" + {else}onload="document.linkform.lf_tags.focus();"{/if} > + + + {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/export.html b/tpl/export.html index 938cbe64f..ff6bd63c3 100644 --- a/tpl/export.html +++ b/tpl/export.html @@ -1,15 +1,24 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + + + {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/import.html b/tpl/import.html index 5fea92d18..207d11ce0 100644 --- a/tpl/import.html +++ b/tpl/import.html @@ -1,21 +1,27 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + + + {include="page.footer"} diff --git a/tpl/includes.html b/tpl/includes.html index e0ad00d55..936cb9289 100644 --- a/tpl/includes.html +++ b/tpl/includes.html @@ -1,10 +1,11 @@ -{$pagetitle} - - - - - - - -{if condition="is_file('inc/user.css')"}{/if} -{if="empty($GLOBALS['disablejquery'])"}{/if} + {$pagetitle} + + + + + + + +{if condition="is_file('inc/user.css')"} {/if} +{if="empty($GLOBALS['disablejquery'])"} + {/if} diff --git a/tpl/install.html b/tpl/install.html index e88606083..e32d77efb 100644 --- a/tpl/install.html +++ b/tpl/install.html @@ -1,21 +1,36 @@ + -{include="includes"}{$timezone_js} + +{include="includes"} +{$timezone_js} + + -
-

Shaarli

-It looks like it's the first time you run Shaarli. Please configure it:
-
-
- - - -{$timezone_html} - - -
Login:
Password:
Page title:
-
-
-{include="page.footer"} +
+

Shaarli

+ It looks like it's the first time you run Shaarli. Please configure it:
+
+
+ + + + + + + + {$timezone_html} + + + + + + +
Login:
Password:
Page title:
+
+
+
+ + {include="page.footer"} diff --git a/tpl/linklist.html b/tpl/linklist.html index 0905e0a68..6fa8069a1 100644 --- a/tpl/linklist.html +++ b/tpl/linklist.html @@ -1,82 +1,109 @@ + -{include="includes"} + +{include="includes"} + + - + - - {include="page.footer"} -{if="empty($GLOBALS['disablejquery'])"} - -{/if} + {include="page.footer"} + {if="empty($GLOBALS['disablejquery'])"} + + {/if} diff --git a/tpl/linklist.paging.html b/tpl/linklist.paging.html index 2b235cb75..7789f1589 100644 --- a/tpl/linklist.paging.html +++ b/tpl/linklist.paging.html @@ -1,20 +1,27 @@ -
-{if="isLoggedIn()"} - - {if="$previous_page_url"} ◄Older {/if} -
page {$page_current} / {$page_max}
- {if="$next_page_url"} Newer► {/if} -
diff --git a/tpl/loginform.html b/tpl/loginform.html index 805a01487..dd722ae92 100644 --- a/tpl/loginform.html +++ b/tpl/loginform.html @@ -1,26 +1,31 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/readme.txt b/tpl/readme.txt index d1002c70c..b3f0f157c 100644 --- a/tpl/readme.txt +++ b/tpl/readme.txt @@ -4,11 +4,16 @@ Any Shaarli page should conform to this RainTPL template: ----------------------------------------------------- -{include="includes"} + +{include="includes"} + + - - You body goes here... - {include="page.footer"} + + + You body goes here... + + {include="page.footer"} ----------------------------------------------------- @@ -20,19 +25,23 @@ If you want to also add something in the page header (in the dark area), do it h Example: "Add new link" form: ----------------------------------------------------- -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + + + {include="page.footer"} ----------------------------------------------------- - diff --git a/tpl/tagcloud.html b/tpl/tagcloud.html index 6918c7be9..b9487ed7f 100644 --- a/tpl/tagcloud.html +++ b/tpl/tagcloud.html @@ -1,15 +1,21 @@ + -{include="includes"} + +{include="includes"} + + - -
-
- {loop="tags"} - {$value.count}{$key|htmlspecialchars} - {/loop} -
-
-{include="page.footer"} + + +
+
+ {loop="tags"} + {$value.count}{$key|htmlspecialchars} + {/loop} +
+
+ + {include="page.footer"} - \ No newline at end of file + diff --git a/tpl/tools.html b/tpl/tools.html index 48ecc97eb..e0509a4c6 100644 --- a/tpl/tools.html +++ b/tpl/tools.html @@ -1,19 +1,34 @@ + -{include="includes"} + +{include="includes"} + + - -{include="page.footer"} + + + {include="page.footer"} - \ No newline at end of file + From 467397922734288a00a408cd52ebd0001ddfa3a2 Mon Sep 17 00:00:00 2001 From: Robert Spencer Date: Mon, 22 Jul 2013 22:51:30 +0200 Subject: [PATCH 6/9] IDs should only be used once, changed to class --- inc/shaarli.css | 4 ++-- tpl/linklist.paging.html | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/inc/shaarli.css b/inc/shaarli.css index cb87a9af9..0b46a23fe 100644 --- a/inc/shaarli.css +++ b/inc/shaarli.css @@ -200,7 +200,7 @@ cursor:pointer; .paging a:visited { color:#ccc; } .paging a:hover { color:#FFFFC9; } .paging a:active { color:#fff; } -#paging_privatelinks { float:left; } +.paging_privatelinks { float:left; } .paging_linksperpage { float:right; padding-right:5px; } .paging_current { display:inline; color:#fff; padding:0 20 0 20; } #paging_older { margin-right:15px; } @@ -450,7 +450,7 @@ a {color:#000!important;text-decoration:none!important;} #searchform_value { width:70% !important; } #tagfilter_value { width:70% !important; } div.qrcode { position:relative; float:left; top:-10px; left:0px; } -#paging_privatelinks { float;none; } +.paging_privatelinks { float;none; } .paging_linksperpage { float:none; margin-bottom:10px; font-size:smaller;} #paging_older,#paging_newer,.paging_linksperpage a { border: 1px solid black; padding:3px 5px 3px 5px; background-color:#666; color:#fff; border-radius: 5px 5px 5px 5px;} .thumbnail { float:none; height:auto; margin: 0px; text-align:center;} diff --git a/tpl/linklist.paging.html b/tpl/linklist.paging.html index 7789f1589..d6cc893c1 100644 --- a/tpl/linklist.paging.html +++ b/tpl/linklist.paging.html @@ -1,6 +1,6 @@
{if="isLoggedIn()"} -