Your push $this->{fields}, "some value"; is not right, either... you'll need to do
push @{$this->{fields}}, "some value";
so the hash value will be treated as an array...

oh, and packages must end in a true value, so many people put a 1; as the last line of their package
                - Ant