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

Lexicons with modified values load endlessly in the manager (ie don't load) - php 5.4.4 #9257

Closed
modxbot opened this Issue Dec 12, 2012 · 4 comments

Comments

Projects
None yet
2 participants
@modxbot

modxbot commented Dec 12, 2012

yogooo created Redmine issue ID 9257

This affects Modx revo 2.2.6. However it is not available in the menu "Affects Revolution Version"

h2. Summary

Lexicons will not load in the manager after any of their values was modified.
Switching from php 5.4.4 to 5.3.14 or 5.4.6 fixes the issue (I haven't tested other php versions).

h2. Step to reproduce

Precondition: run all under php 5.4.4

  • Install a fresh version of modx on a fresh database
  • Login
  • Top Menu => System > Lexicon Management
  • Pick a lexicon by changing "Topic" or "Language"
  • Modify a value
  • Reload the page (you're back to the Core:Default:en lexicon)
  • Access the lexicon you modified

h2. Observed

  • The loading spinner shows up and never goes away
  • The lexicon never loads
  • The POST request seems to be valid (sniffed it and didn't spot anything unusual - see screenshots)

h2. Expected

The lexicon loads and is displayed as usual

h2. Workaround

Switch to php 5.3.14 or 5.4.6 (I haven't tested other php versions)

@opengeek

This comment has been minimized.

Show comment
Hide comment
@opengeek

opengeek Dec 12, 2012

Member

opengeek submitted:

I'm running PHP 5.4.4 for all MODX and xPDO testing and cannot reproduce this at all. Check permissions on your cache files. Perhaps something is resetting permissions when you change PHP versions? Very odd...

Member

opengeek commented Dec 12, 2012

opengeek submitted:

I'm running PHP 5.4.4 for all MODX and xPDO testing and cannot reproduce this at all. Check permissions on your cache files. Perhaps something is resetting permissions when you change PHP versions? Very odd...

@modxbot

This comment has been minimized.

Show comment
Hide comment
@modxbot

modxbot Dec 12, 2012

yogooo submitted:

Here's what I have on my local machine (mamp, php 5.4.4. and 5.3.14) :

drwxr-xr-x@ cache
drwxr-xr-x lexicon_topics and all sub-directories

  • r w - r - - r - - action.cache.php and all similar files

They don't change when switching from one php version to the other.

On the production server under 5.4.6, 'group' doesn't have any permissions.

modxbot commented Dec 12, 2012

yogooo submitted:

Here's what I have on my local machine (mamp, php 5.4.4. and 5.3.14) :

drwxr-xr-x@ cache
drwxr-xr-x lexicon_topics and all sub-directories

  • r w - r - - r - - action.cache.php and all similar files

They don't change when switching from one php version to the other.

On the production server under 5.4.6, 'group' doesn't have any permissions.

@modxbot

This comment has been minimized.

Show comment
Hide comment
@modxbot

modxbot Dec 12, 2012

yogooo submitted:

Here's a partial diff of the php conf. 5.3.14 against 5.4.4. I removed what's irrelevant/has been deprecated. Do you spot anything that might cause the issue?

modxbot commented Dec 12, 2012

yogooo submitted:

Here's a partial diff of the php conf. 5.3.14 against 5.4.4. I removed what's irrelevant/has been deprecated. Do you spot anything that might cause the issue?

@opengeek

This comment has been minimized.

Show comment
Hide comment
@opengeek

opengeek Dec 28, 2012

Member

opengeek submitted:

I have now been able to reproduce this; it is a new strict notice due to improper use of array_diff_assoc(). I am correcting this for 2.2.7-pl.

Member

opengeek commented Dec 28, 2012

opengeek submitted:

I have now been able to reproduce this; it is a new strict notice due to improper use of array_diff_assoc(). I am correcting this for 2.2.7-pl.

enigmatic-user pushed a commit to enigmatic-user/revolution that referenced this issue Feb 13, 2014

This issue was closed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment