Skip Menu |
 

This queue is for tickets about the RT-Extension-Assets-Import-CSV CPAN distribution.

Report information
The Basics
Id: 120685
Status: open
Priority: 0/
Queue: RT-Extension-Assets-Import-CSV

People
Owner: Nobody in particular
Requestors: T.Pletter [...] rewe-group.at
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



CC: 'Czapek Roland - Firma' <Roland.Czapek [...] Nexellence.at>, REWE OBS <OBS [...] rewe-group.at>
Subject: [OBS-SUBPROJ #4034] old tablenames like RTxAssets and RTxCatalog / Extend variable sep_char module Text::CSV_XS
Date: Wed, 22 Mar 2017 08:47:32 +0000
To: "'bug-RT-Extension-Assets-Import-CSV [...] rt.cpan.org'" <bug-RT-Extension-Assets-Import-CSV [...] rt.cpan.org>
From: Pletter Thomas <T.Pletter [...] rewe-group.at>
Download (untitled) / with headers
text/plain 2.1k
Hello, Is this module still maintained ? A bug was reported at Do 28. Jan 2016, 06:11:39 from jvandewege : @ I'm currently working on a plan to upgrade our RT instance to the coming RT-4.4 release and want to import a lot of assets @ from an old system. I've upgrade my test system to 4.4RC3 and discovered that the latest version of this extension isn't adapted @to that version. This extension still uses the old tablenames like RTxAssets and RTxCatalog We have also the same problem. We changed temporarily the sequences und table_names to Assets and tried it again. It seems ok ? Is there a possibility to minimal extend RT-Extension-Assets-Import-CSV ? We need another sep_char because in columns exist the sep_char "," The module Text::CSV_XS has the possibility like : $csv = Text::CSV_XS->new({ 'quote_char' => '"', 'escape_char' => '"', 'sep_char' => ',', 'binary' => 0 }); Here are my changes ( change tablenames and sequence, extend sep_char ), i am not a experienced perl programmer, so you can do it better : Kind regards, Thomas Show quoted text
______________________________________ HINWEIS: Alle von Gesellschaften der REWE International AG elektronisch uebermittelten Auskuenfte, Mitteilungen und Erklaerungen sind vertraulich und rechtlich unverbindlich. Verpflichtungen von Gesellschaften der REWE International AG beduerfen der schriftlichen Einigung in saemtlichen Vertragspunkten sowie der Unterfertigung durch das vertretungsbefugte Organ. Im Falle einer Fehluebermittlung ersuchen wir um Benachrichtigung sowie um Vernichtung und Beachtung, dass jegliche Form der Verwendung und Weitergabe der Inhalte an Dritte unzulaessig ist. NOTICE: All notices, declarations and explanations transferred electronically from REWE International AG companies are confidential and not legally binding. Obligations of REWE International AG companies arise only when written agreement to all contractual points has been confirmed in writing with the signature of an authorized representative. If this e-mail was sent by mistake, please inform us immediately. Furthermore we also request you to destroy it and use neither the contents nor disclose them in any manner to third parties.
Download CSV.pm
text/x-perl 15k

Message body is not shown because sender requested not to inline it.

Download rt-assets-import-csv
application/octet-stream 3.1k

Message body not shown because it is not plain text.

Download (untitled) / with headers
text/plain 1.7k
On Wed Mar 22 05:03:08 2017, T.Pletter@rewe-group.at wrote: Show quoted text
> Hello,
Hi Thomas, Show quoted text
> Is this module still maintained ?
Yes it is :) Show quoted text
> A bug was reported at Do 28. Jan 2016, 06:11:39 from jvandewege : > > @ I'm currently working on a plan to upgrade our RT instance to the > coming RT-4.4 release and want to import a lot of assets > @ from an old system. I've upgrade my test system to 4.4RC3 and > discovered that the latest version of this extension isn't adapted > @to that version. This extension still uses the old tablenames like > RTxAssets and RTxCatalog > > We have also the same problem. We changed temporarily the sequences > und table_names to Assets and tried it again. > It seems ok ?
Sorry about that. As I just mentioned in the other ticket, we did have a 4.4 compat release over a year ago, but we missed a place which affects only Pg and Oracle. We've just today released version 2.2 that fixes that issue. Show quoted text
> Is there a possibility to minimal extend RT-Extension-Assets-Import- > CSV ? > We need another sep_char because in columns exist the sep_char "," > > The module Text::CSV_XS has the possibility like : > > $csv = Text::CSV_XS->new({ > 'quote_char' => '"', > 'escape_char' => '"', > 'sep_char' => ',', > 'binary' => 0 > }); > > Here are my changes ( change tablenames and sequence, extend sep_char > ), i am not a experienced perl programmer, > so you can do it better :
Thanks! I would like to have that feature too :) We could take it one step further, and have the extension consult a new RT_SiteConfig setting which would be a hash of options to pass to Text::CSV_XS. That way, other users can adjust things in addition to sep_char. Could you try implementing that? Show quoted text
> Kind regards, > Thomas
Thanks, Shawn


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.