ljg3

In the instance of the hhvm driver in object-cache.php, the code is missing the database select if: (added at line 380)

if ( isset( $redis[ 'database' ] ) ) {
  $this->redis->select( $redis[ 'database' ] );
}

Putting that in fixed the DB select issue. Salting works fine as is in 1.1.

This also occurs if you bump into your API rate limit, which is a deal breaker, as it can unexpectedly take the site down.

Thanks, somehow I wasn’t getting notifications on the the posts for this plugin so I missed this. 1.1 has a similar list, with a little bit of javascript/css for styling it.

Thanks!
-Lew

That feature was added in 1.1

Thanks!
-Lew

Update. This is better solution, as it takes into account whether TinyMCE has been loaded or not. Here’s the patch:

wysiwyg.patch

You can apply it by moving that file into the more-fields plugin folder and running:

patch -p0 < wysiwyg.patch

Commenting out line 80 in more-fields-settings-object.php did the trick for me.

//wp_tiny_mce( false ); // true gives you a stripped down version of the editor

I haven’t had a chance to completely back-trace this, but at least it un-breaks TinyMCE, and the additional fields all still seem to function properly. Use at your own risk!

Do we know if there are plans to incorporate this patch into the next release? I’ve had to patch the plugin as well to support CPT permalinks.

Same issue I’ve had when trying the cache. My initial thought for a fix is to disable the object cache if you’re logged into the backend (which is the default setting, I believe, in other caching plugins like W3 Total Cache). Before I do that, I thought I’d see if anyone in this thread was able to get a root cause or fix?

To fix the plugin, you have to replace the version of jQuery UI used. So, in the /js/ folder of the plugin use a 1.8 version with UI.Sortable included. For those in a rush, you can just use this file and drop it into that folder:

https://goettner.code.s3.amazonaws.com/jquery-ui.min.js

Unfortunately, the server I’m playing on now is firewalled into our local IPs only. I’ve actually switched the fix because it looks like when you retrieve the site_url, that trailing slashes are intentionally stripped. So here’s the new diff I’ve been using (on lib/W3/Plugin/Cdn.php):

@@ -785,7 +785,7 @@

                                         $dst_basename = basename($dst);
                                         $dst_dirname = dirname($dst);
-                                        $dst_url = sprintf('%s%s/%s', $site_url, $upload_url, $dst_basename);
+                                        $dst_url = sprintf('%s/%s/%s', $site_url, $upload_url, $dst_basename);

                                         w3_mkdir($dst_dirname, 0755, ABSPATH);

@@ -971,7 +971,7 @@
                     if (preg_match_all($regexp, $post_content, $matches, PREG_SET_ORDER)) {
                         foreach ($matches as $match) {
                             $old_url = $match[2];
-                            $new_url = sprintf('%s%s%s', $site_url, $upload_info['upload_url'], $match[5]);
+                            $new_url = sprintf('%s/%s%s', $site_url, $upload_info['upload_url'], $match[5]);
                             $post_content = str_replace($old_url, $new_url, $post_content);

                             $results[] = array(

I was having the same problem. I took a look through the code and it seems as though the way the upload functionality was written that it skips the original files now. Here’s the code in particular, lib/W3/Plugin/Cdn.php starting at line 655:

if (isset($metadata['file'])) {
	$files = array_merge($files, $this->get_metadata_files($metadata));
} elseif (!empty($post->file)) {
	$file = $this->normalize_attachment_file($post->file);
	$local_file = $upload_info['upload_dir'] . '/' . $file;
	$remote_file = $upload_info['upload_url'] . '/' . $file;
	$files[$local_file] = $remote_file;
}

I rewrote the logic so it always includes the main file, in addition to files found in the metadata:

if (!empty($post->file)) {
	$file = $this->normalize_attachment_file($post->file);
	$local_file = $upload_info['upload_dir'] . '/' . $file;
	$remote_file = $upload_info['upload_url'] . '/' . $file;
	$files[$local_file] = $remote_file;
}
if (isset($metadata['file'])) {
	$files = array_merge($files, $this->get_metadata_files($metadata));
}

After that, I had a much larger group of files uploaded, and everything seems to be working correctly. You could also make the change in the function that processes the metadata looking for files, but I’ll leave that decision up to the plugin author.

-Lew