Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

Conversation

kynx
Copy link
Contributor

@kynx kynx commented Jan 9, 2016

  • XCache adapter now checks version > 3.1.0 when running under CLI
  • fixed broken internalSetItem() with empty namespace

Don't got the chops to get travis building the xcache extension so the tests can run - doesn't look like it's avail from pecl (I get mine from brew). Anyone got any ideas?

- XCache adapter now checks version > 3.1.0 when running under CLI
- fixed broken internalSetItem() with empty namespace
@kynx
Copy link
Contributor Author

kynx commented Jan 9, 2016

Ach, since github doesn't link from the titles, this addresses #58

@@ -341,10 +341,17 @@ protected function internalSetItem(& $normalizedKey, & $value)
{
$options = $this->getOptions();
$namespace = $options->getNamespace();
$prefix = ($options === '') ? '' : $namespace . $options->getNamespaceSeparator();
$prefix = ($namespace === '') ? '' : $namespace . $options->getNamespaceSeparator();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch 👍

@marc-mabe marc-mabe self-assigned this Jan 9, 2016
@kynx
Copy link
Contributor Author

kynx commented Jan 16, 2016

Fixed in #65

@kynx kynx closed this Jan 16, 2016
@marc-mabe marc-mabe added the bug label Feb 12, 2016
@marc-mabe marc-mabe added this to the 2.6.2 milestone Feb 12, 2016
@marc-mabe marc-mabe modified the milestones: 2.7.0, 2.6.2 Apr 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants