# [joomla#27196] Unable to add a trashed Content Language again. Thanks

Michael

administrator/components/com_admin/sql/updates/mysql/1.7.4-2011-11-19.sql

@@ -0,0 +1,2 @@
+ALTER TABLE `#__languages` ADD UNIQUE `idx_image` (`image`);
+ALTER TABLE `#__languages` ADD UNIQUE `idx_langcode` (`lang_code`);

administrator/components/com_languages/models/language.php

@@ -168,7 +168,7 @@ public function save($data)
 
 		// Store the data
 		if (!$table->store()) {
-			$this->setError($this->_db->getErrorMsg());
+			$this->setError($table->getError());
 			return false;
 		}
 

administrator/language/en-GB/en-GB.lib_joomla.ini

@@ -164,6 +164,9 @@ JLIB_DATABASE_ERROR_INVALID_LOCATION="%s: :setLocation - Invalid location"
 JLIB_DATABASE_ERROR_INVALID_NODE_RECURSION="%s: :move Failed - Cannot move the node to be a child of itself"
 JLIB_DATABASE_ERROR_INVALID_PARENT_ID="Invalid parent id."
 JLIB_DATABASE_ERROR_LANGUAGE_NO_TITLE="The language should have a title"
+JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_IMAGE="A content language already exists with this Image Prefix"
+JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_LANG_CODE="A content language already exists with this Language Tag"
+JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_SEF="A content language already exists with this URL Language Code"
 JLIB_DATABASE_ERROR_LOAD_DATABASE_DRIVER="Unable to load Database Driver: %s"
 JLIB_DATABASE_ERROR_MENUTYPE_EMPTY="Menu type empty"
 JLIB_DATABASE_ERROR_MENUTYPE_EXISTS="Menu type exists: %s"

installation/CHANGELOG

@@ -27,6 +27,9 @@ $ -> Language fix or change
 - -> Removed
 ! -> Note
 
+03-Dec-2011 Jean-Marie Simonet
+ # [#27196] Unable to add a trashed Content Language again. Thanks Michael
+
 03-Dec-2011 Christophe Demko
  ^ [#26778] Update TinyMCE to 3.4.7
  # [#27255] Fix some E_STRICT warnings

installation/sql/mysql/joomla.sql

@@ -180,7 +180,7 @@ CREATE TABLE `#__categories` (
   `checked_out` int(11) unsigned NOT NULL default '0',
   `checked_out_time` datetime NOT NULL default '0000-00-00 00:00:00',
   `access` integer unsigned NOT NULL default '0',
-  `params` TEXT NOT NULL, 
+  `params` TEXT NOT NULL,
   `metadesc` varchar(1024) NOT NULL COMMENT 'The meta description for the page.',
   `metakey` varchar(1024) NOT NULL COMMENT 'The meta keywords for the page.',
   `metadata` varchar(2048) NOT NULL COMMENT 'JSON encoded metadata properties.',
@@ -545,6 +545,8 @@ CREATE TABLE `#__languages` (
   `ordering` int(11) NOT NULL default '0',
   PRIMARY KEY  (`lang_id`),
   UNIQUE `idx_sef` (`sef`),
+  UNIQUE `idx_image` (`image`),
+  UNIQUE `idx_langcode` (`lang_code`),
   INDEX `idx_ordering` (`ordering`)
 )  DEFAULT CHARSET=utf8;
 

language/en-GB/en-GB.lib_joomla.ini

@@ -164,6 +164,9 @@ JLIB_DATABASE_ERROR_INVALID_LOCATION="%s: :setLocation - Invalid location"
 JLIB_DATABASE_ERROR_INVALID_NODE_RECURSION="%s: :move Failed - Cannot move the node to be a child of itself"
 JLIB_DATABASE_ERROR_INVALID_PARENT_ID="Invalid parent id."
 JLIB_DATABASE_ERROR_LANGUAGE_NO_TITLE="The language should have a title"
+JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_IMAGE="A content language already exists with this Image Prefix"
+JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_LANG_CODE="A content language already exists with this Language Tag"
+JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_SEF="A content language already exists with this URL Language Code"
 JLIB_DATABASE_ERROR_LOAD_DATABASE_DRIVER="Unable to load Database Driver: %s"
 JLIB_DATABASE_ERROR_MENUTYPE_EMPTY="Menu type empty"
 JLIB_DATABASE_ERROR_MENUTYPE_EXISTS="Menu type exists: %s"

libraries/joomla/database/table/language.php

@@ -49,4 +49,39 @@ public function check()
 
 		return true;
 	}
+
+	/**
+	 * Overrides JTable::store to check unique fields.
+	 *
+	 * @param   boolean  $updateNulls  True to update fields even if they are null.
+	 *
+	 * @return  boolean  True on success.
+	 *
+	 * @since   11.4
+	 */
+	public function store($updateNulls = false)
+	{
+		// Verify that the sef field is unique
+		$table = JTable::getInstance('Language', 'JTable');
+		if ($table->load(array('sef' => $this->sef)) && ($table->lang_id != $this->lang_id || $this->lang_id == 0))
+		{
+			$this->setError(JText::_('JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_SEF'));
+			return false;
+		}
+
+		// Verify that the image field is unique
+		if ($table->load(array('image' => $this->image)) && ($table->lang_id != $this->lang_id || $this->lang_id == 0))
+		{
+			$this->setError(JText::_('JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_IMAGE'));
+			return false;
+		}
+
+		// Verify that the language code is unique
+		if ($table->load(array('lang_code' => $this->lang_code)) && ($table->lang_id != $this->lang_id || $this->lang_id == 0))
+		{
+			$this->setError(JText::_('JLIB_DATABASE_ERROR_LANGUAGE_UNIQUE_LANG_CODE'));
+			return false;
+		}
+		return parent::store($updateNulls);
+	}
 }