Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Update Memcache.php #278

Merged
merged 1 commit into from

2 participants

@galvani

this shouldn't get commited, don't you test it?

@galvani galvani Update Memcache.php
this shouldn't get commited, don't you test it?
e81c059
@rmccue
Owner

Sorry about this. There's currently no tests for the Memcache cache handler. I'd absolutely love some if you've got any ideas! :)

Thanks for the patch! :cake:

@rmccue rmccue merged commit 351b92e into simplepie:master

1 check passed

Details default The Travis build passed
@galvani

I will write the memcache test and the redis test if you accept the patch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Mar 22, 2013
  1. @galvani

    Update Memcache.php

    galvani authored
    this shouldn't get commited, don't you test it?
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  library/SimplePie/Cache/Memcache.php
View
2  library/SimplePie/Cache/Memcache.php
@@ -95,7 +95,7 @@ public function __construct($location, $name, $type)
'prefix' => 'simplepie_',
),
);
- $this->options = SimplePie_Misc::merge_array_recursive($this->options, SimplePie_Cache::parse_URL($location);
+ $this->options = SimplePie_Misc::array_merge_recursive($this->options, SimplePie_Cache::parse_URL($location));
$this->name = $this->options['extras']['prefix'] . md5("$name:$type");
Something went wrong with that request. Please try again.