Mateusz Bieniek
Mateusz Bieniek
@gggeek currently yes, but it is almost completed.
@gggeek can you please point me to the part of the code you are talking about?
@gggeek the "midnight" part was a bad case of copy&paste from Date FieldType PR's description - sorry for that. In legacy dates/datetimes were always stored as timestamps - this PR...
@gggeek I get your point, but still - it is out of the scope of this PR. Nevertheless, I'm taking it into the account and do my best to improve...
@glye about docs: yes I agree. I got confused because those are from the "root PR" and I couldn't pinpoint where I could put some doc blocks here :wink:
Script for updating DB: https://github.com/ezsystems/ezpublish-kernel/pull/2499
@pkamps @pbek After some checking I think I found the root of the problem: We get `$Param['Email']` from URI (`eZURI`), which is initated here: https://github.com/ezsystems/ezpublish-legacy/blob/master/kernel/private/classes/ezpkernelweb.php#L1083 using `eZSys::requestURI()`. `eZSys::requestURI()` is initaited...
I have to test it, but it seems like https://github.com/mugoweb/ezpublish-legacy/pull/119 alongside with https://github.com/ezsystems/ezpublish-legacy/pull/1360 seems like a good approach and should work.
Hi @vidarl - Please take a second look - I've probably updated the text when you were writing your review. But I agree that this might not be the best...