Ticket #24 (closed defect: fixed)

Opened 6 years ago

Last modified 4 years ago

undefined charset if dbh is passed to new storage

Reported by: dom Owned by: dom
Priority: normal Milestone:
Component: wiki-toolkit Version: svn
Keywords: hackfestsummer2007-reviewed Cc:

Description (last modified by dom) (diff)

[from  http://rt.cpan.org/Public/Bug/Display.html?id=26761]

Wiki::Toolkit::Store::Pg->new( dbh => $dbh, charset => 'utf-8' );

In the above code, if you either specify charset or not it is ignored and undefined so charset_encode/charset_decode functions don't have anything to work with.

This is the chage i've made to Wiki::Toolkit::Store::Database version 0.27 as a quick fix. I'm not sure if this is the best fix though.

if ( $args{dbh} ) {
$self->{_dbh} = $args{dbh};
$self->{_external_dbh} = 1; # don't disconnect at DESTROY time
+ $self->{_charset} = $args{charset} || "iso-8859-1";
} else {
die "Must supply a dbname" unless defined $args{dbname};

Thank you.

Change History

Changed 6 years ago by dom

Code should have been:

if ( $args{dbh} ) {
$self->{_dbh} = $args{dbh};
$self->{_external_dbh} = 1; # don't disconnect at DESTROY time
+ $self->{_charset} = $args{charset} || "iso-8859-1";
} else {
die "Must supply a dbname" unless defined $args{dbname};

Changed 6 years ago by dom

  • owner changed from nick to Nobody

Changed 6 years ago by ilmari

  • keywords hackfestsummer2007-reviewed added
  • owner changed from Nobody to ilmari
  • status changed from new to assigned

Changed 5 years ago by dom

  • description modified (diff)

Changed 4 years ago by dom

  • owner changed from ilmari to dom
  • status changed from assigned to new

Changed 4 years ago by dom

  • status changed from new to assigned

Changed 4 years ago by dom

  • status changed from assigned to closed
  • resolution set to fixed

(In [481]) Configure charset correctly when passing in an existing $dbh (fixes #24)

Note: See TracTickets for help on using tickets.