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

Fixes $cache key names where needed #79

Merged
merged 1 commit into from Sep 12, 2014

Conversation

@amenadiel
Copy link
Contributor

amenadiel commented Jul 17, 2014

Apparently some keys in the $cache array changed names, so nothing was displayed in the control panel for user variables.

This pull request fixes the following keys:

  • key instead of info
  • atime instead of access_time
  • dtime instead of deletion_time
  • ctime instead of creation_time
  • nhits instead of num_hits

The replacement is done only where needed. i.e. if the array keys somehow go back to their old form, they'll be left untouched.

Apparently some keys in the $cache array changed names, so nothing was displayed in the control panel for user variables.

This pull request fixes the following keys:

* key instead of info
* atime instead of access_time
* dtime instead of deletion_time
* ctime instead of creation_time
* nhits instead of num_hits

The replacement is done only where needed. i.e. if the array keys somehow go back to their old form, they'll be left untouched.
@amenadiel

This comment has been minimized.

Copy link
Contributor Author

amenadiel commented Jul 30, 2014

@krakjoe bump? 😀

@thestanislav

This comment has been minimized.

Copy link

thestanislav commented Sep 10, 2014

May be related to this #56 ?

@amenadiel

This comment has been minimized.

Copy link
Contributor Author

amenadiel commented Sep 11, 2014

Yup, it fixes that issue.

krakjoe added a commit that referenced this pull request Sep 12, 2014
Fixes $cache key names where needed
@krakjoe krakjoe merged commit e2c7cd9 into krakjoe:simplify Sep 12, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
@krakjoe

This comment has been minimized.

Copy link
Owner

krakjoe commented Sep 12, 2014

Sorry about the delay, so sorry ...

I'll be working on apcu in the next couple of days, I'm getting to it ... if all is good now I'll push a release ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.