benvanstaveren / Mojolicious-Plugin-Mongodb

Use MongoDB in Mojolicious
http://search.cpan.org/dist/Mojolicious-Plugin-Mongodb/
2 stars 2 forks source link

Fixed "no_OPTION" in register & updated docs. #2

Closed SailingYYC closed 10 years ago

SailingYYC commented 11 years ago

The for loop uses a named variable but the conditional in the loop uses $_.

Strangely enough one of my apps passes this without complaint, the other warns of 3 uses of undef in a concatenation.

SailingYYC commented 10 years ago

@benvanstaveren, Any chance you can version bump master to v1.16 and push out to CPAN?

Most appreciated, Colin

benvanstaveren commented 10 years ago

Ugh, I knew I was forgetting something!

benvanstaveren commented 10 years ago

Pushed as 1.16

SailingYYC commented 10 years ago

No worries and thanks. I think that closes off the other 2 open issues as well.