----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105503/#review15622 -----------------------------------------------------------
This review has been submitted with commit 051dd78586c31413616101fa5e1cbfc5232a3118 by Friedrich W. H. Kossebau to branch master. - Commit Hook On July 10, 2012, 2:49 p.m., Friedrich W. H. Kossebau wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/105503/ > ----------------------------------------------------------- > > (Updated July 10, 2012, 2:49 p.m.) > > > Review request for Calligra and Inge Wallin. > > > Description > ------- > > Some calligratests like > interoperability/wordprocessing/oowriter/oow_embede_audio.odt fail on a bad > xml:id for the draw:plugin element. > Reason is that PluginShape currently has a property xmlid, which only gets > set on loading. On saving the content of that property is written, even if > empty (like for the file above). > > Attached patch fixes this by doing like the rest(?) of Calligra, dropping any > read xmlid and generating in any case a new one on saving. > (Also fixes a bad i18n-with-parameter call and deprecated QString::null > usages, or should those be in separate commits?) > > Okay to backport to 2.5? > > PS: In general I think Calligra should try to remember the read xml-ids, it > kind-of spoils the idea of ids if those are completely new after saving, even > for untouched elements. But that is not in the scope of this fix, which only > cares for writing valid ODF. > > > Diffs > ----- > > plugins/pluginshape/PluginShape.h 38c67db > plugins/pluginshape/PluginShape.cpp c1e19ec > > Diff: http://git.reviewboard.kde.org/r/105503/diff/ > > > Testing > ------- > > > Thanks, > > Friedrich W. H. Kossebau > >
_______________________________________________ calligra-devel mailing list calligra-devel@kde.org https://mail.kde.org/mailman/listinfo/calligra-devel
