Hello Christoph,
thank you for writjng the module in the first place! Don't worry about taking
time to reply - I know how it is when I am busy with my dayjob.
I'd be very happy to adopt Spreadsheet::ReadSXC. If you still have your CPAN /
PAUSE credentials, you can assign me ("CORION") as Co-maintainer or primary
maintainer as per
http://www.cpan.org/modules/04pause.html#takeover
If you don't have (and want) your CPAN credentials anymore, please send a mail
to [email protected] (also in CC) that they should assign me (co-)maintainership
or move the module to the user ADOPTME.
Looking forward to the module,
-max
Am 17. September 2019 00:41:17 MESZ schrieb Christoph Terhechte
<[email protected]>:
>Hello Max,
>
>Thanks for your email. I’m surprised anyone is still using that module.
>It was written for a much earlier version of Open Office many years
>ago. I have since abandoned programming more or less.
>
>I’d be happy to apply the patches you suggest, but I’m afraid my job is
>keeping me so busy that it would take quite a while to do so. If you’d
>like to “adopt” the module, I’d be happy to let you take over. Not
>having been on CPAN for more than a decade (at least), I am not aware
>of any necessary steps to take, but if you are, please let me know.
>
>Best regards,
>
>Christoph
>
>> On 30. Aug 2019, at 11:10, Max Maischein <[email protected]> wrote:
>>
>> Hello Christoph,
>>
>> thank you very much for writing Spreadsheet::ReadSXC! The module
>helps
>> me to conveniently read spreadsheet files from OpenOffice and
>LibreOffice.
>>
>> I format some money values with a currency but still need to get at
>the
>> numbers as numbers. The attached patches implement reading of raw
>values
>> for currency cells. Along the way I also found that ODS changed the
>> attribute name from SXC files for the raw date and raw time values.
>>
>> It would be great if you find the time to review the patches and
>apply
>> them. If you don't want to maintain the module anymore, I'm also
>willing
>> to make a new release with these fixes.
>>
>> The patches in detail are:
>>
>> 0001 - simplify the test suite using is_deeply in favour of cmp_array
>>
>> This is a minor style change but not really necessary for the new
>> functionality. It just uses functionality that comes with Perl.
>>
>> 0002 - eliminate @ISA in favour of using the Exporter 'import'
>feature
>> directly
>>
>> This is a minor style change but not really necessary for the new
>> functionality. It just uses functionality that comes with Perl.
>>
>> 0003 - switch reading the XML file from a global filehandle to a
>lexical
>> filehandle
>>
>> This is a minor style change but not really necessary for the new
>> functionality. It just uses functionality that comes with Perl.
>>
>> 0004 - support ODS formatted cell values
>>
>> 0005 - test for reading a file twice
>>
>> This is mostly a sanity check that I didn't break that functionality
>>
>> 0006 - add the option to read the raw values of formatted numbers
>>
>> This is the meat of the changes.
>>
>> Thanks again for writing this module,
>>
>> -max
>>
>>
><0001-Eliminate-array_cmp-in-favour-of-is_deeply.patch><0002-Switch-to-Exporter-5-usage.patch><0003-Switch-XML-reading-to-a-lexical-filehandle.patch><0004-Support-ODS-formatted-cell-values-as-well.patch><0005-Add-test-for-reading-the-same-file-twice.patch><0006-Add-StandardCurrency-option-and-tests.patch><0007-Update-Changes.patch>
--
Diese Nachricht wurde von meinem Android-Mobiltelefon mit K-9 Mail gesendet.
