Skip to content

Commit

Permalink
Add some more TODO comments about the named csv idea
Browse files Browse the repository at this point in the history
  • Loading branch information
rwstauner committed Dec 22, 2012
1 parent 2fd2bbe commit bf1ab58
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions lib/App/csv2sqlite.pm
Expand Up @@ -78,6 +78,7 @@ sub getopt {
$p->getoptions($opts,
'csv_files|csv-file|csvfile|csv=s@',
# TODO: 'named_csv_files=s%'
# or maybe --csv and --named should be subs that append to an array ref to keep order?
'csv_options|csv-opt|csvopt|o=s%',
# TODO: tableloader options like 'drop' or maybe --no-create
'loader_options|loader-opt|loaderopt|l=s%',
Expand Down Expand Up @@ -109,6 +110,10 @@ sub load_tables {
file => $file,
);

# TODO: This could work but i hate the escaping thing.
# Allow table=file (use "=file" for files with an equal sign).
#if( $file =~ /^([^=:]*)[=:](.+)$/ ){ $opts{name} = $1 if $1; $opts{file} = $2; }

DBIx::TableLoader::CSV->new(
%opts,
dbh => $self->dbh,
Expand Down

0 comments on commit bf1ab58

Please sign in to comment.