[patch #6358] Text import patch pre-review take 4

[Ben Pfaff]

Follow-up Comment #15, patch #6358 (project pspp):

>...when I import my /etc/passwd file, it assigns the password 
>field with format MONTH3. Presumably this is because it's 
>classifying "x" as the roman numeral ten. The year 10 AD is of 
>course not representable in PSPP, so it produces funny results.
>
>Perhaps it should only treat things which could be roman numerals 
>as dates if they're in recent history? 

PSPP only allows months to be written as roman numerals, not years.  The
MONTH format represents a month but not a date; that is, a normal field of
type MONTH would contain something like "JANUARY", and your field that
contains "x" is interpreted as October.  So there's no year or day to make it
recent or long ago.

I've never actually encountered a data file that represents months with roman
numerals, even though SPSS supports it, so I think what I'll do here is
disable recognition of a roman numeral on its own as a month, only supporting
them when they are part of a larger date format, e.g. "1 x 1978" in EDATE
format (not that I've seen anyone do that either).

>If I override the allocated format to string, then on import, I 
>get an error: 

Oops.  Will fix.

    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?6358>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/