| # |
Feb 2nd 2018, 13:53 |
hmic |
good hint |
| # |
Feb 2nd 2018, 13:49 |
 admad |
do `$this->helpers = $this->helpers()->normalizeArray($this->helpers);` first :slightly_smiling_face: |
| # |
Feb 2nd 2018, 13:47 |
hmic |
so i need to check all the elements, if numerically indexed, and remove if i want to load an alias for them :/ |
| # |
Feb 2nd 2018, 13:47 |
hmic |
still the HtmlHelper is in the $this->helpers like: [0 => 'Html'] while the alias, which should overwrite/remove the former one is ['Html' => 'BootstrapUI.Html'] of course |
| # |
Feb 2nd 2018, 13:46 |
hmic |
thats what i fixed already |
| # |
Feb 2nd 2018, 13:46 |
admad |
"CrudView.CrudView" would be split internally and the alias would still be "CrudView" in helper registry |
| # |
Feb 2nd 2018, 13:41 |
hmic |
even worse, if i change that to use an alias too, it complains Html is loaded already later on, as this is loaded in the default appview, unaliased... :[ |
| # |
Feb 2nd 2018, 13:40 |
hmic |
looks good at first sight, but it's actually not. as my alias for CrudView is "CrudView", while it gets loaded as "CrudView.CrudView" in the method |
| # |
Feb 2nd 2018, 13:39 |
hmic |
i changed _setupHelpers to set $this->helpers accordingly, not using $this->loadHelper() anymore |
| # |
Feb 2nd 2018, 13:39 |
hmic |
admad: it's not as easy as i thought it should be still :/ |
| # |
Feb 2nd 2018, 13:23 |
hmic |
all of them are locking correctly, all of them allow for multiple webservers and loadbalancing too, both of which (local) files don't |
| # |
Feb 2nd 2018, 13:23 |
k4t |
yes, memcache should solve it |
| # |
Feb 2nd 2018, 13:22 |
hmic |
change the session config to use database (worst case), or any cache, memcache is more than common, redis works too |
| # |
Feb 2nd 2018, 13:22 |
hmic |
thats a php thing and known forever |
| # |
Feb 2nd 2018, 13:22 |
hmic |
k4t: file sessions are not concurrent. |
| # |
Feb 2nd 2018, 13:21 |
k4t |
for storing sessions |
| # |
Feb 2nd 2018, 13:21 |
k4t |
instead of forcing users to use some kind of cache engines |
| # |
Feb 2nd 2018, 13:21 |
k4t |
to be honest I understand the problem, and security reasons behind session renew logic, but is it possible to fix it somehow on framework side? |
| # |
Feb 2nd 2018, 13:20 |
k4t |
as such thing can happen |
| # |
Feb 2nd 2018, 13:19 |
k4t |
does it means that keeping session in files in completely useless? |
| # |
Feb 2nd 2018, 13:19 |
hmic |
slaps k4t |
| # |
Feb 2nd 2018, 13:18 |
k4t |
@admad as a workaround I started writing directly to $_SESSION :( |
| # |
Feb 2nd 2018, 13:17 |
hmic |
let me try |
| # |
Feb 2nd 2018, 13:17 |
hmic |
makes sense |
| # |
Feb 2nd 2018, 13:17 |
admad |
k4t: your problem seems to happen because the session file if still locked from previous request when new request is made. I would suggest using a cache engine backed session instead of default file based sessions |
| # |
Feb 2nd 2018, 13:14 |
admad |
*_setupHelpers() |
| # |
Feb 2nd 2018, 13:14 |
admad |
hence my suggestion to update _setHelpers() to modify $helpers instead of loading helpers directly through loadHelper() |
| # |
Feb 2nd 2018, 13:13 |
admad |
hmic: loadHelper() does, but CrudView loads it's custom helpers through initialize() which is called before loadHelpers() hence the aliases already loaded are ignored |
| # |
Feb 2nd 2018, 13:10 |
k4t |
I found some similar problems on GitHub: https://github.com/cakephp/cakephp/issues/5988 Unfortunetelly they do not solve my case. |
| # |
Feb 2nd 2018, 13:10 |
k4t |
I am using Cake 3.1.14, PHP 5.6. Session cookies are handled by Cake, but the same problem appear when I change it to php. |
| # |
Feb 2nd 2018, 13:09 |
k4t |
I discovered that error is caused by Cake session renew logic, exactly by that line in Session.php Cake file: session_regenerate_id(true); When I set session_regenerate_id flag to false then everything is fine (becuase session file is not deleted). This happens when I am writing to session via $this->Auth->setUser($user) (but it is random); |
| # |
Feb 2nd 2018, 13:09 |
k4t |
Guys I have big issue with session in CakePHP, when I keep opening multiple tabs in browser fast then session is lost and following error is displayed: "session_regenerate_id(): Session object destruction failed". Here is more detailed log: https://gist.github.com/K4T/6584c957355da498c18d260abd1a4ddd |
| # |
Feb 2nd 2018, 13:05 |
hmic |
actually, maybe even loadHelper should consider viewbuilder itself... |
| # |
Feb 2nd 2018, 12:59 |
admad |
you can make a PR :slightly_smiling_face: |
| # |
Feb 2nd 2018, 12:59 |
admad |
hmic: having helper config through viewBuilder take precedence can be achieved if _setupHelpers() was changed to check and modify $helpers property instead of using loadHelper() |
| # |
Feb 2nd 2018, 12:58 |
hmic |
i think it's an easy fix to make that work |
| # |
Feb 2nd 2018, 12:58 |
hmic |
i would have expected that way to work to inject my overwritten crudviewhelper into crudview... |
| # |
Feb 2nd 2018, 12:57 |
hmic |
i usually don't either. but it seems to be the way to inject helpers in a custom viewclass |
| # |
Feb 2nd 2018, 12:54 |
admad |
hmic: yes i have extended CrudView and overridden _setupHelpers() method. I personally don't use viewBuilder in controller to manage helpers |
| # |
Feb 2nd 2018, 12:48 |
max775 |
yeaaah I noticed that... damn stuff... everything is working besides this shitty date XD ok ill try to get everything to UTC and see if its working thenˆ^ thanks |
| # |
Feb 2nd 2018, 12:46 |
openmail_slack |
The DB should always be UTC. Then make sure that you only put UTC times into the DB (convert or run the server on UTC). Then output the date on the page by the setting your user (in his settings) or you as app creator have chosen. Internally in your you only want to talk in UTC, everything else is PITA |