1
0
mirror of https://github.com/silverstripe/silverstripe-translatable synced 2024-10-22 09:05:59 +00:00

Added check to see if the DataObject actually has translation before iterating over.

This commit is contained in:
Gus King 2013-10-14 01:56:07 -04:00
parent 242acac7dd
commit e5b95f9bb2

@ -1069,22 +1069,24 @@ class Translatable extends DataExtension implements PermissionProvider {
_t('Translatable.EXISTING', 'Existing translations'), _t('Translatable.EXISTING', 'Existing translations'),
3 3
)); ));
if(!$tab->fieldByName('existingtrans')) { if (!$tab->fieldByName('existingtrans')) {
$existingTransHTML = '<ul>'; $existingTransHTML = '<ul>';
foreach($this->getTranslations() as $existingTranslation) { if ($existingTranslations = $this->getTranslations()) {
if($existingTranslation && $existingTranslation->hasMethod('CMSEditLink')) { foreach ($existingTranslations as $existingTranslation) {
if ($existingTranslation && $existingTranslation->hasMethod('CMSEditLink')) {
$existingTransHTML .= sprintf( $existingTransHTML .= sprintf(
'<li><a href="%s">%s</a></li>', '<li><a href="%s">%s</a></li>',
Controller::join_links( Controller::join_links(
$existingTranslation->CMSEditLink(), $existingTranslation->CMSEditLink(),
'?locale='.$existingTranslation->Locale '?locale=' . $existingTranslation->Locale
), ),
i18n::get_locale_name($existingTranslation->Locale) i18n::get_locale_name($existingTranslation->Locale)
); );
} }
} }
}
$existingTransHTML .= '</ul>'; $existingTransHTML .= '</ul>';
$tab->push(new LiteralField('existingtrans',$existingTransHTML)); $tab->push(new LiteralField('existingtrans', $existingTransHTML));
} }
} }
} }