The commit 754060b rewrites a large part of the code, but makes the recursion of the next_change() method inoperative. If someone has any idea of how to do that (or to do better!), please don't hesitate to propose a pull request!
The goal is to get the true next change for a field like Mo-Fr 00:00-24:00. Currently, for a test from a Monday, it will return 24:00 on Monday, it should return 24:00 on Friday.
rezemika
commented
6 years ago
The following concerns the
new-parsingbranch.The commit 754060b rewrites a large part of the code, but makes the recursion of the
next_change()method inoperative. If someone has any idea of how to do that (or to do better!), please don't hesitate to propose a pull request!The goal is to get the true next change for a field like
Mo-Fr 00:00-24:00. Currently, for a test from a Monday, it will return 24:00 on Monday, it should return 24:00 on Friday.Here is the concerned method: https://github.com/rezemika/humanized_opening_hours/blob/754060bca1ccef7d1aa8e6e6e8107c981725feee/humanized_opening_hours/main.py#L348-L377