From marc@infomarc.info Thu Jul 14 20:56:29 2011 From: Marc Delisle To: developers@phpmyadmin.net Subject: Re: [Phpmyadmin-devel] XML export Date: Thu, 14 Jul 2011 16:57:47 -0400 Message-ID: <4E1F584B.5020009@infomarc.info> In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1323335243236139440==" --===============1323335243236139440== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Le 2011-07-14 15:43, Piotr Przybylski a =C3=A9crit : > 2011/7/14 Marc Delisle: >> Le 2011-07-14 09:24, Piotr Przybylski a =C3=A9crit : >>> 2011/7/14 Marc Delisle: >>>> Le 2011-07-13 18:28, Piotr Przybylski a =C3=A9crit : >>>>> 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.: >>>>> >>>>> >>>>> value >>>>> ... >>>>> >>>>> ... >>>>>
>>>>> >>>> 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 doin= g 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=20 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! --=20 Marc Delisle http://infomarc.info --===============1323335243236139440==--