search

dalgibbard / citrix_xenserver_patcher

Auto / Manual Patching tool for Citrix XenServer Boxes
Do What The F*ck You Want To Public License
142 stars 45 forks source link

Simplify the null patchlist check #37

Closed dalgibbard closed 9 years ago

dalgibbard commented 9 years ago

This line (currently line 522 in my code stream): if inst_patch_list == [] or inst_patch_list == "" or inst_patch_list == ['']: needs condensing to the null check which actually is needed. This was a dirty fix whilst I wasn't sure.

dalgibbard commented 9 years ago

Added something in debug to monitor for this; and trap the right one. It's not important anyway, so closing until it pops up.