Review Request: honor the suffix of the filename the user selected when exporting from odt to (x)html

[mojtaba shahi]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/107676/#review23368
-----------------------------------------------------------

Ship it!


Ship It!

- mojtaba shahi


On Dec. 12, 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/107676/
> -----------------------------------------------------------
> 
> (Updated Dec. 12, 2012, 2:49 p.m.)
> 
> 
> Review request for Calligra, Inge Wallin and mojtaba shahi.
> 
> 
> Description
> -------
> 
> Currently the suffix is ignored and always .xhtml is used. While .xhtml might be the correct suffix for the content which is currently created by the export filter, still the user should be more right. If she/he wants something else, it should be delivered.
> 
> Attached patch changes the behaviour to use the suffix of the filename passed to the export filter for all files that are generated.
> It's not the optimal patch. I had no real idea how to inject the user-choosen suffix just for the case of one file, as with (x)html export. Perhaps you have a better idea.
> But at least it works :)
> 
> Okay to commit to 2.6 and port to master?
> 
> 
> Diffs
> -----
> 
>   filters/words/epub/FileCollector.h e705026 
>   filters/words/epub/FileCollector.cpp b25bfa6 
>   filters/words/epub/OdtHtmlConverter.cpp ac8dcc4 
>   filters/words/epub/exporthtml.cpp d657490 
> 
> Diff: http://git.reviewboard.kde.org/r/107676/diff/
> 
> 
> Testing
> -------
> 
> export as html to a file with custom suffix uses that suffix
> export to epub uses xhtml as before internally
> 
> 
> Thanks,
> 
> Friedrich W. H. Kossebau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20121212/c611db9f/attachment.htm>