Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Weird error (probably due to PHP update) with wrong date-period returning default php objects instead of SimpUtils #121

Open
PandaHugMonster opened this issue Oct 30, 2022 · 0 comments

Comments

@PandaHugMonster
Copy link
Owner

$dt_period = DT::walk('2022-02-24', '2022-03-30', '1 day');
// IMP  Stop the war! Save Ukraine! Slava Ukraini!
foreach ($dt_period as $day) {
  /** @var \DateTime $day - this is wrong type, it must return SimpUtils version */
  $this->assertContains("{$day->date}", $dt_expected);
}
PandaHugMonster added a commit that referenced this issue Oct 30, 2022
Un-related weird test error related to wrong date-time object returned from date-period. Ticket here: #121
PandaHugMonster added a commit that referenced this issue Oct 30, 2022
* Small adjustments of the documentation inside of basic.php

* Progress on PHPDOC preparation for basic.php

* Progress on PHPDOC preparation for basic.php
Really good one.

Additionally a bug with DateTime and wrong TZ choosing is resolved.

* php-simputils-115 Ee improvement

Un-related weird test error related to wrong date-time object returned from date-period. Ticket here: #121

* php-simputils-115 Ee improvement #115

Working nice code, tests might be improved later even further.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant