| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 2.0.4 |
| Bug #2631 | Translation2_Admin::add() produces duplicates | ||
|---|---|---|---|
| Submitted: | 2004-10-27 14:11 UTC | ||
| From: | ylf at xung dot org | Assigned: | quipo |
| Status: | Closed | Package: | Translation2 |
| PHP Version: | 4.3.4 | OS: | Linux Debian (testing) |
| Roadmaps: | (Not assigned) | ||
[2004-10-27 14:11 UTC] ylf at xung dot org
Description:
------------
In bug #2570 I asked for an Admin:update() method, which you did include. Thanks for that.
But, the add() method performs in a pretty inconsistent manner. It should sometimes perform an update.
The point is that the string id and page id couple should form a unique key : there can't be two rows in the i18n table with both the same string id and page id, or the table is inconsistent. And this is not respected.
Currently add() means 'add a new string id', but one may think add() also means : 'add a new translation for this string id'. Of course, the documentation could state that for this later case, update() is the right way.
But I believe you have to do some extra checking, otherwise the tables may get inconsistent when someone adds twice the same string id.
After checking the reproduce code you may think that the closest solution to this issue would be to use the mysql REPLACE statement, but it won't work :
"Values for all columns are taken from the values specified in the REPLACE statement. Any missing columns are set to their default values, just as happens for INSERT."
(see: http://dev.mysql.com/doc/mysql/en/REPLACE.html)
So, to keep tables consistent, I guess you'll have to
- either issue more queries,
- or do some comparison between the cache and what's to be add()ed
- or use an sql constraint as "INDEX full_id (page_id,id)" which will result in a DB error if there's a duplicate row (the lightest and smartest to me, but is it portable ?)
Reproduce code:
---------------
<?php
error_reporting (E_ALL);
require_once 'PEAR.php';
require_once 'DB.php';
require_once 'Translation2/Admin.php';
$dsn = '';
$db =& DB::connect($dsn);
// Starting with an empty db : tables langs and i18n do not exist
$db->query('drop table if exists langs, i18n');
$tr =& Translation2_Admin::factory('DB', $db);
if (PEAR::isError($tr)) exit($tr->getMessage());
$newLang = array(
'lang_id' => 'en',
'table_name' => 'i18n',
'name' => 'english',
'meta' => 'some meta info',
'error_text' => 'not available');
$tr->createNewLang($newLang);
$newLang = array(
'lang_id' => 'fr',
'table_name' => 'i18n',
'name' => 'french',
'meta' => 'some meta info',
'error_text' => 'not available');
$tr->createNewLang($newLang);
// this works fine :
$tr->add('string1','page1',array('en' => 'string 1', 'fr' => 'texte 1'));
// now let's add an english only translation
$tr->add('string2','page1',array('en' => 'string 2'));
// and, subsequently, the french translation for the same string id
$tr->add('string2','page1',array('fr' => 'chaine 2'));
// and even, the english translation again for the same string id
$tr->add('string2','page1',array('en' => 'string 2'));
echo "<html><body><pre>";
echo "\n\n----------------------------------------\n";
echo "table : i18n\n";
echo "----------------------------------------\n";
$res =& $db->getAll("select * from i18n",array(),DB_FETCHMODE_ASSOC);
print_r ($res);
echo "</pre></body></html>";
?>
Expected result:
----------------
----------------------------------------
table : i18n
----------------------------------------
Array
(
[0] => Array
(
[page_id] => page1
[id] => string1
[en] => string 1
[fr] => texte 1
)
[1] => Array
(
[page_id] => page1
[id] => string2
[en] => string 2
[fr] => chaine 2
)
)
Actual result:
--------------
----------------------------------------
table : i18n
----------------------------------------
Array
(
[0] => Array
(
[page_id] => page1
[id] => string1
[en] => string 1
[fr] => texte 1
)
[1] => Array
(
[page_id] => page1
[id] => string2
[en] => string 2
[fr] =>
)
[2] => Array
(
[page_id] => page1
[id] => string2
[en] =>
[fr] => chaine 2
)
[3] => Array
(
[page_id] => page1
[id] => string2
[en] => string 2
[fr] =>
)
)
Comments
[2004-10-27 19:10 UTC] quipo
This bug has been fixed in CVS.
In case this was a documentation problem, the fix will show up at the
end of next Sunday (CET) on pear.php.net.
In case this was a pear.php.net website problem, the change will show
up on the website in short time.
Thank you for the report, and for helping us make PEAR better.