[Rt-devel] patches for spreadsheet export

[Kevin Falcone]

Please follow my Reply-To, I read the list and don't need two copies.

On Thu, Dec 16, 2010 at 06:46:27PM +0100, Christian Loos wrote:
> Am 16.12.2010 18:00, schrieb Kevin Falcone:
> > On Thu, Dec 16, 2010 at 11:56:35AM -0500, Kevin Falcone wrote:
> >>> utf-8 didn't work on Windows XP with Excel 2003.
> >>
> >> Does it work on more modern Excels?  This seems like a really odd
> >> patch.  I wonder if it would even work on Macs with Numbers.
> > 
> > Forgetting client issues for a minute, how would this patch work on
> > actual utf8 data (such as asian languages)
> OK forget about the first patch.
> I will try to test utf8 on more modern excel. If this still fails maybe
> it makes sense to have a config value where you can override utf8
> encoding if it doesn't work for someone like me.

That sounds like a much less destructive option.

> >> Is there a reason not to use RT's default scrubber?
> html/Elements/ScrubHTML don't scrub all html tags but in this case we
> have to scrub all.
> >>
> >>> +<%ONCE>
> >>> +my $scrubber = HTML::Scrubber->new();
> >>> +$scrubber->default(0);
> >>> +$scrubber->comment(0);
> >>> +</%ONCE>

Is the problem here that we're getting HTML blown into the
spreadsheet, or that you're getting broken html?  If HTML is leaking,
then yes, we want some sort of scrub.

-kevin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <http://lists.bestpractical.com/pipermail/rt-devel/attachments/20101216/41bcf357/attachment.pgp>