From 43fc7f93f2dc75001e6aeebd0e446976aae5485c Mon Sep 17 00:00:00 2001
From: Gus King <gus@mediadevils.com>
Date: Mon, 14 Oct 2013 01:56:07 -0400
Subject: [PATCH 1/2] Fix for odd error

---
 code/model/Translatable.php | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/code/model/Translatable.php b/code/model/Translatable.php
index 124f52d..ba0d963 100755
--- a/code/model/Translatable.php
+++ b/code/model/Translatable.php
@@ -1071,19 +1071,21 @@ class Translatable extends DataExtension implements PermissionProvider {
 				));
 				if(!$tab->fieldByName('existingtrans')) {
 					$existingTransHTML = '<ul>';
-					foreach($this->getTranslations() as $existingTranslation) {
-						if($existingTranslation && $existingTranslation->hasMethod('CMSEditLink')) {
+					if ($existingTranslations = $this->getTranslations()) { // TODO PR
+						foreach ($existingTranslations as $existingTranslation) {
+							if ($existingTranslation && $existingTranslation->hasMethod('CMSEditLink')) {
 							$existingTransHTML .= sprintf(
 								'<li><a href="%s">%s</a></li>',
 								Controller::join_links(
 									$existingTranslation->CMSEditLink(),
-									'?locale='.$existingTranslation->Locale
+										'?locale=' . $existingTranslation->Locale
 								),
 								i18n::get_locale_name($existingTranslation->Locale)
 							);
 						}
 					}
-					$existingTransHTML .= '</ul>';	
+					} // TODO PR
+					$existingTransHTML .= '</ul>';
 					$tab->push(new LiteralField('existingtrans',$existingTransHTML));
 				}
 			}

From 94785f34f61acae5f9b9be5224edb16ab1406c98 Mon Sep 17 00:00:00 2001
From: Gus King <gus@mediadevils.com>
Date: Mon, 14 Oct 2013 02:13:59 -0400
Subject: [PATCH 2/2] Fixed indentation and removed //todo comments.

---
 code/model/Translatable.php | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

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