return utcnow() - then > datetime.timedelta(seconds=seconds)
98 + continue
log message?
105 + if _vbd_ref["userdevice"] == "1":
Is there a risk of userdevice not being defined?
Perhaps, if _vbd_ref.get("userdevice", None) == "1":
173 + if last_ran:
174 + if not utils.is_then_greater(last_ran, timeout * 60 * 60):
175 + # Do not run. Let's bail.
176 + return
177 + else:
178 + # Update the time tracker and proceed.
179 + self.poll_rescue_last_ran = utils.utcnow()
180 + else:
181 + # We need a base time to start tracking.
182 + self.poll_rescue_last_ran = utils.utcnow()
183 + return
I try to keep my returns together to keep the if blocks smaller
173 + if not last_ran:
181 + # We need a base time to start tracking.
182 + self.poll_rescue_last_ran = utils.utcnow()
183 + return
174 + if not utils.is_then_greater(last_ran, timeout * 60 * 60):
175 + # Do not run. Let's bail.
176 + return
177 + # Update the time tracker and proceed.
179 + self.poll_rescue_last_ran = utils.utcnow()
9 +def is_then_ greater( then, seconds): timedelta( seconds= seconds) :
40 + if utcnow() - then > datetime.
41 + return True
42 + else:
43 + return False
Beyond the naming as matt pointed out, how about
return utcnow() - then > datetime. timedelta( seconds= seconds)
98 + continue
log message?
105 + if _vbd_ref[ "userdevice" ] == "1":
Is there a risk of userdevice not being defined? get("userdevice ", None) == "1":
Perhaps, if _vbd_ref.
173 + if last_ran: then_greater( last_ran, timeout * 60 * 60): rescue_ last_ran = utils.utcnow() rescue_ last_ran = utils.utcnow()
174 + if not utils.is_
175 + # Do not run. Let's bail.
176 + return
177 + else:
178 + # Update the time tracker and proceed.
179 + self.poll_
180 + else:
181 + # We need a base time to start tracking.
182 + self.poll_
183 + return
I try to keep my returns together to keep the if blocks smaller
173 + if not last_ran: rescue_ last_ran = utils.utcnow()
181 + # We need a base time to start tracking.
182 + self.poll_
183 + return
174 + if not utils.is_ then_greater( last_ran, timeout * 60 * 60):
175 + # Do not run. Let's bail.
176 + return
177 + # Update the time tracker and proceed. rescue_ last_ran = utils.utcnow()
179 + self.poll_
No tests?