Thanks for the report Jason. We’ll look into it.
Jason, is this still an issue? I’ve just tested this, and I do get a warning about a deprecated function. But not a fatal error.
I’m getting the same issue. Here’s the error I get on site creation –
Catchable fatal error: Argument 1 passed to WP_HTTP_Requests_Response::__construct() must be an instance of Requests_Response, none given, called in /home/techiec8/public_html/wp-content/plugins/multisite-cloner/multisite-cloner.php on line 355 and defined in /home/techiec8/public_html/wp-includes/class-wp-http-requests-response.php on line 45
I’ve also noticed that the image files from my default site have not been getting copied since the update. The site sort of replicates, but it’s missing a lot 🙁
I haven’t been able to test this yet because my dev server is in the process of being set up (and I don’t dare test this live) but what if lines 354-355 were replaced with this line of code –
$_tmp = new ReflectionClass(get_class( $data ));
I haven’t worked with reflection classes before but I think I understand what this is doing and why this would work even though the current code (below) doesn’t
$dataClass = get_class( $data );
$_tmp = new $dataClass( );
I’m assuming the problem is with trying to create a class from a variable value rather than being able to create it directly with the class name. Hopefully this is helpful in fixing this problem. My customers are not exactly impressed that their first welcome to my network is a fatal error :/
Hat tip to StackExchange for the possible solution.
I finally got my dev site put together – the fix I proposed a couple weeks ago works for me.
@manuel Razzari May be it’s time for update. I’m really waiting. 🙂
Thank for your tests @techiechic, we’ll look into it ASAP.
I’m also getting the exact same fatal error, and would love to see an update. It’s worked for me in the past, but something must have changed with a wordpress update.
Thanks for the heads up. We’ll be releasing an update this week.
I found a fix for this error. I just forgot to share it here with everyone months ago.
Open up multisite-cloner.php and either comment out or remove line 352 to 362
/*
// Submitted by Tina Matter
elseif ( is_object( $data ) ) {
$dataClass = get_class( $data );
$_tmp = new $dataClass( );
foreach ( $data as $key => $value ) {
$_tmp->$key = cloner_recursive_unserialize_replace( $from, $to, $value, false );
}
$data = $_tmp;
unset( $_tmp );
}
*/
I’ve just pushed an update, please test and let me know. You’ll need to re download the plugin, as I didn’t trigger a new public release yet.
@techiechic re:
> I’ve also noticed that the image files from my default site have not been getting copied since the update. The site sort of replicates, but it’s missing a lot
I wasn’t able to reproduce this one. If it’s still an issue for you, please open a new ticket and we’ll look into it.
Hi @manuelrazzari
Thanks for the update! I tested the new release (version 0.2.2) and it worked smoothly and accurately. The only miss, though small, I noticed is that the “Tagline” of the master blog was not copied into the new blog created.
I will post here if I’ll notice any other miss.
Thank you very much for the update.
Regards!
@t2m that’s weird, I’ve just double-checked and the tagline works as expected.
I’m marking this as resolved as I’ve just published 0.2.2 with fixes to the issue originally reported here. (And thanked you all in the changelog)
If the tagline issue persists, please open a new ticket. Much thanks everyone!