[Phpmyadmin-devel] XML export

[Marc Delisle]

Le 2011-07-14 15:43, Piotr Przybylski a écrit :
> 2011/7/14 Marc Delisle<marc at infomarc.info>:
>> Le 2011-07-14 09:24, Piotr Przybylski a écrit :
>>> 2011/7/14 Marc Delisle<marc at infomarc.info>:
>>>> Le 2011-07-13 18:28, Piotr Przybylski a écrit :
>>>>> Hi,
>>>>>
>>>>> Is XML export used at all? It was missing data escaping in many
>>>>> places, and structure (table, function, ...) export was just broken. I
>>>>> fixed it a bit, but there are still some problems that remain:
>>>>> - escaping of table/db names and data in other encodings than ISO-8859-1
>>>>> - it uses SHOW CREATE TABLE without any required fixes
>>>>>
>>>>> File format itself is interesting. I don't expect XML Schema for it to
>>>>> exist, but at least namespace usage should be consistent.
>>>>>
>>>>> I think fixing this and rewriting output format to be more consistent
>>>>> should appear on Tasks for junior developers. And maybe extending it
>>>>> to output data in more concise format, eg.:
>>>>> <table name="">
>>>>> <row>
>>>>> <column_name>value</column_name>
>>>>> ...
>>>>> </row>
>>>>> ...
>>>>> </table>
>>>>>
>>>> Piotr,
>>>> XML export is probably used, how can we know?
>>>
>>> Maybe nobody needs DDL in it or we have too few people using it to
>>> report bugs :)
>>>
>>>> In your testing, was it just a problem in master or also in 3.4.x?
>>>>
>>>
>>> Also in 3.4.x (STABLE). Go to database `evil'"*/>` and try to export
>>> table `evil'"*/>` with all options selected. You will get only data
>>> (no structure) and invalid XML
>>
>> Can you backport your fixes to QA_3_4?
>>
>
> Yes. Just to be sure I don't break anything - what's the proper way of doing it?
> 1. Create a local branch traacking origin/QA_3_4
> 2. cherry-pick my commits to master
> 3. push
> ?
>

This should work. Usually you should have started with QA_3_4, like 
explained on
http://wiki.phpmyadmin.net/pma/Git#Committing_fixes_to_several_branches.

The big mistake to avoid is merging master to QA_3_4!

-- 
Marc Delisle
http://infomarc.info