Changeset 270 for veekun/trunk/lib/Vee.pm

Timestamp:

07/31/07 17:26:10 (3 years ago)

Author:

eevee

Message:

Attempted fix for the duplicate key error.

Files:

• veekun/trunk/lib/Vee.pm (modified) (2 diffs)

veekun/trunk/lib/Vee.pm

@@ -65 +65 @@
 
 {
+    # WARNING: HERE BE DRAGONS, AND BY DRAGONS I MEAN HACKS
+    # TODO: WHAT THE FUCK CLEAN THIS GARBAGE UP WITH PLUGINS OR SOMETHING
+
     no strict 'refs';
 
@@ -73 +76 @@
         $self->reset;
         return not $first;
+    };
+
+    # DBIx::Class's find_or_create is incredibly stupid and allows for race
+    # conditions with critical code like this.
+    # This is MySQL-specific!  Also it is a horrendous hack.
+    # CAVEAT EMPTOR: may break if you upgrade
+    *{'Catalyst::Plugin::Session::Store::DBIC::Delegate::session'} = sub {
+        my ($self, $key) = @_;
+
+        my $row = $self->_session_row;
+
+        unless ($row) {
+            eval {
+                $row = $self->model->find_or_create({ $self->id_field => $key });
+            }
+            if ($@ and $@ =~ /Duplicate entry/) {
+                $row = $self->model->create({ $self->id_field => $key });
+            } else {
+                die $@;
+            }
+            $self->_session_row($row);
+        }
+
+        return $row;
     };
 }