----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/105503/#review15608 -----------------------------------------------------------
plugins/pluginshape/PluginShape.cpp <http://git.reviewboard.kde.org/r/105503/#comment12165> I think we should also use the xmlid type KoElementReference::Counter here as UUID is just a waste for something like this. plugins/pluginshape/PluginShape.cpp <http://git.reviewboard.kde.org/r/105503/#comment12166> I think this should be changed to !name.isEmpty() as you no longer pass an null string here. - Thorsten Zachmann On July 10, 2012, 12:39 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, 12:39 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
