Fix DBALStorage to not ignore exceptions

[enumag]

Is there some reason to catch all exceptions? If there is a problem with the storage I want to know about it, not silently ignore it.

[Ocramius]

Requires testing. /cc @EmanueleMinotto

[EmanueleMinotto]

@enumag you're right, exceptions should be thrown there as well, anyway without tests a merge won't be safe (I'll cover them asap if you can't).

[enumag]

@EmanueleMinotto I don't have time for that now... Also I've noticed that the DBALStorage completely ignores the $storageName argument. Maybe it should be used as a prefix for the key?

Anyway I've duplicated the class to my project for the time being (without the try-catch blocks).

[EmanueleMinotto]

@enumag yes, I'm reviewing everything in these days

[enumag]

@EmanueleMinotto I'm also wondering if there should be some sort of support for transactions (when inserting/updateing multiple values). Not sure if that's relevant to other storages than sql database.

[EmanueleMinotto]

@enumag it would be great if you could open an issue for every thought :) because $storageName seems a bug, I'm still not sure about transactions and no one of these is related to DBALStorage exceptions

[enumag]

Ok, I'll open separate issues. :-)

[enumag]

All done. You can delete the unrelated commens from this issue if you want.

[enumag]

@EmanueleMinotto Any progress?

[EmanueleMinotto]

@enumag not yet, I'm busy with some things + work, I'll take a look asap