preaction / Minion-Backend-mysql

MySQL backend for the 🐙 Minion job runner
Other
7 stars 14 forks source link

Fix use of uninitialized value warning. #20

Closed cromedome closed 5 years ago

cromedome commented 5 years ago

When trying to assemble the history for use in the Minion admin dashboard, the MySQL backend produces the following warning:

Use of uninitialized value in numeric ne (!=) at .../Minion/Backend/mysql.pm line 58.

This patch ensures that the warning never occurs by ensuring there is data available for comparison before attempting to make a comparison.

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 98.429% when pulling 00f9f823586ec220bf2bd3f9fea47f3ce1ff4e4f on cromedome:master into b21a729a02db11e7c56ca87277361b2f0a051331 on preaction:master.

preaction commented 5 years ago

Thanks! I'll ship this out right now