Upgrade 3.4 to 4.6.9 error converting UTF8

[dutchsnowden]

immediately after this, I click begin conversion (I did this many many times and worked perfectly) but now I get this error:

Error: count(): Argument #1 ($var) must be of type Countable|array, null given File: /system/Convert/Convert.php Line: 552

on a otherwise page. Nothing else.

[dutchsnowden]

This is how conf_global.php looks like:

[dutchsnowden]

                                if ( $batchBytes > ( $this->_data['max_allowed_packet'] - ( ( $this->_data['max_allowed_packet'] / 100 ) * ( \count( $batch ) * 0.3 ) ) ) )
                                {
                                        /* Remove last row */
                                        $nextBatch = array_pop( $batch );

                                        static::log( "Max packet hit with " . $batchBytes . 'b with ' . \count( $batch ) . ' rows' );
                                }

                                /* If we have moved the only row from $batch to $nextbatch, then process that now */
                                if ( \count( $nextBatch ) and ! \count( $batch ) )
                                {
                                        $batch     = $nextBatch;
                                        $nextBatch = array();
                                }

 

[dutchsnowden]

Can I skip somehow UTF8 conversion and do it later? Convert manually?

 

[Stuart Silvester]

2 minutes ago, dutchsnowden said:

Can I skip somehow UTF8 conversion and do it later? Convert manually?

No, if you skip UTF8 conversion your entire site will be unreadable.

Try changing

if ( \count( $nextBatch ) and ! \count( $batch ) )

to

if ( \is_array( $nextBatch ) and \count( $nextBatch ) and ! \count( $batch ) )

 

[dutchsnowden]

No difference, exact same error. (LE: I copied the above text before last edit, once I copied the last version and pasted it inthere, it worked flawlessly)

 

Edited January 5, 2022 by dutchsnowden

[Stuart Silvester]

Make sure that your code is the same as mine, I edited it almost immediately after I posted it, but perhaps after you saw it.

[dutchsnowden]

if ( \is_array( $nextBatch ) and \count( $nextBatch ) and ! \count( $batch ) )

this

[Stuart Silvester]

Your screenshot doesn't show the same issue, did you click convert and continue the UTF8 conversion?

[dutchsnowden]

Keep fingers crossed. AND HUGE THANK YOU!

For me it's now or never. I cannot go to sleep until I upgrade.

 

5 minutes ago, Stuart Silvester said:

Your screenshot doesn't show the same issue, did you click convert and continue the UTF8 conversion?

Changed as you instructed, hit refresh at convert and it started as normal.

Not sure, I did something wrong? I definitely hope not.

 

Edited January 5, 2022 by dutchsnowden

[IveLeft...]

Good luck with the live upgrade 👌

[dutchsnowden]

Thanks to all the team for such prompt help!

🤩

 

You are all rockstars!

Just now, Muddy Boots said:

Good luck with the live upgrade 👌

Thanks, I need it!

[Randy Calvert]

Good luck!

[dutchsnowden]

So far so good!

[dutchsnowden]

I don't recall upgrade to look like this, however it still goes on. Is this normal?

[Randy Calvert]

Yes. On large conversions, that can happen. I believe it’s moved to URL refreshing instead of the in-line refreshing. 

This could happen with other large reprocessing activities like if you ever need to rebuild the search cache or rebuild all topics etc. 

You’re still in good shape!

[Marc]

Yes, thats normal when an item is taking a long while to run