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

Fixes #600 - Shaarli's title is not set with the new config manager

Browse source 
- Fixed title config key
  - Page title (in head tag) is no longer set through the config manager
This commit is contained in:
ArthurHoaro 2016-07-19 18:03:09 +02:00
parent 0c4c7ae818
commit 97ef33bb72
2 changed files with 7 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
application/PageBuilder.php
View file

@ -68,18 +68,12 @@ private function initialize()
$this->tpl->assign('source', index_url($_SERVER)); $this->tpl->assign('source', index_url($_SERVER));
$this->tpl->assign('version', shaarli_version); $this->tpl->assign('version', shaarli_version);
$this->tpl->assign('scripturl', index_url($_SERVER)); $this->tpl->assign('scripturl', index_url($_SERVER));
$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 ($this->conf->exists('general.title')) { $this->tpl->assign('pagetitle', $this->conf->get('general.title', 'Shaarli'));
$this->tpl->assign('pagetitle', $this->conf->get('general.title'));
}
if ($this->conf->exists('general.header_link')) { if ($this->conf->exists('general.header_link')) {
$this->tpl->assign('titleLink', $this->conf->get('general.header_link')); $this->tpl->assign('titleLink', $this->conf->get('general.header_link'));
} }
if ($this->conf->exists('pagetitle')) { $this->tpl->assign('shaarlititle', $this->conf->get('general.title', 'Shaarli'));
$this->tpl->assign('pagetitle', $this->conf->get('pagetitle'));
}
$this->tpl->assign('shaarlititle', $this->conf->get('title', 'Shaarli'));
$this->tpl->assign('openshaarli', $this->conf->get('security.open_shaarli', false)); $this->tpl->assign('openshaarli', $this->conf->get('security.open_shaarli', false));
$this->tpl->assign('showatom', $this->conf->get('feed.show_atom', false)); $this->tpl->assign('showatom', $this->conf->get('feed.show_atom', false));
$this->tpl->assign('hide_timestamps', $this->conf->get('privacy.hide_timestamps', false)); $this->tpl->assign('hide_timestamps', $this->conf->get('privacy.hide_timestamps', false));

12
index.php
View file

@ -1675,10 +1675,7 @@ function buildLinkList($PAGE,$LINKSDB, $conf, $pluginManager)
$keys[] = $key; $keys[] = $key;
} }
// If there is only a single link, we change on-the-fly the title of the page.
if (count($linksToDisplay) == 1) {
$conf->set('pagetitle', $linksToDisplay[$keys[0]]['title'] .' - '. $conf->get('general.title'));
}
// Select articles according to paging. // Select articles according to paging.
$pagecount = ceil(count($keys) / $_SESSION['LINKS_PER_PAGE']); $pagecount = ceil(count($keys) / $_SESSION['LINKS_PER_PAGE']);
@ -1740,9 +1737,10 @@ function buildLinkList($PAGE,$LINKSDB, $conf, $pluginManager)
'links' => $linkDisp, 'links' => $linkDisp,
'tags' => $LINKSDB->allTags(), 'tags' => $LINKSDB->allTags(),
); );
// FIXME! temporary fix - see #399.
if ($conf->exists('pagetitle') && count($linkDisp) == 1) { // If there is only a single link, we change on-the-fly the title of the page.
$data['pagetitle'] = $conf->get('pagetitle'); if (count($linksToDisplay) == 1) {
$data['pagetitle'] = $linksToDisplay[$keys[0]]['title'] .' - '. $conf->get('general.title');
} }
$pluginManager->executeHooks('render_linklist', $data, array('loggedin' => isLoggedIn())); $pluginManager->executeHooks('render_linklist', $data, array('loggedin' => isLoggedIn()));