pluck
pluck 4.7 out now, download it today!

pluck support forums

for all your questions about the easiest CMS on the planet

You are not logged in.

Announcement

New registrations are disabled for some spam problems. New registrations will be avaliable soon. If you want to help to keep forum away from spam - contact us.
Please post your bug reports on Launchpad.

#1 27-03-2010 14:56:25

a_bach
developer/module guru
From: Poland
Registered: 24-11-2008
Posts: 439
Website

launchpad

I was trying to add new bug in launchpad.

Unexpected form data

Launchpad doesn't understand the form data submitted in this request.

Bug is about: theme_pagetitle sometimes dont return valid title in subpages. Create 'das' supage. It will show as instead of 'das'.

Last edited by a_bach (27-03-2010 14:57:46)

Offline

#2 27-03-2010 15:16:03

a_bach
developer/module guru
From: Poland
Registered: 24-11-2008
Posts: 439
Website

Re: launchpad

I think

$page_title = trim(implode(' · ', $titles));

instead of

 foreach ($titles as $title)
				$final_title .=  ' · '.$title;

			$page_title = ltrim($final_title, '· ');

in get_pagetitle() will do the job.

One more think:

function theme_sitetitle() {
	echo SITE_TITLE;
}

So why dont delete this function and in theme.php type:

<?php echo SITE_TITLE; ?>

Offline

#3 31-03-2010 13:26:29

Sander
project webmaster
From: The Netherlands
Registered: 14-06-2007
Posts: 942
Website

Re: launchpad

a_bach wrote:

One more think
function theme_sitetitle() {
    echo SITE_TITLE;
}
So why dont delete this function and in theme.php type:
<?php echo SITE_TITLE; ?>

I think it's better to keep that as it is now, for backward compatibility with older themes. If we delete the function, al lot of custom themes will break in the next version.

About the other fix you mention; if you've tested it thouroughly, I think you can commit the fix to the main branch.

Offline

#4 07-04-2010 09:15:50

a_bach
developer/module guru
From: Poland
Registered: 24-11-2008
Posts: 439
Website

Re: launchpad

Sander wrote:

I think it's better to keep that as it is now, for backward compatibility with older themes. If we delete the function, al lot of custom themes will break in the next version.

About the other fix you mention; if you've tested it thouroughly, I think you can commit the fix to the main branch.

Fix commited.

And I was thinkig about functions... So dont delete those function, but start using prefefined variables in new themes.

A_Bach

Offline

Board footer

Powered by FluxBB