Print Page - ManageLanguages.php (Line 396): Illegal string offset 'writable'

Title: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: m4z on April 12, 2019, 04:55:33 AM

SMF2.1rc2 on Debian 8, PHP5.4(?), Postgresql 9.4, ...

Type of error: General
2: Illegal string offset 'writable'
https://redacted/~beta-smf/index.php?action=admin;area=languages;sa=downloadlang;did=33313;[hexstring]
/home/beta-smf/public_html/Sources/ManageLanguages.php (Line 396)

Immediately followed by another error:

Type of error: General
2: Cannot use a scalar value as an array
https://redacted/~beta-smf/index.php?action=admin;area=languages;sa=downloadlang;did=33313;[hexstring]
/home/beta-smf/public_html/Sources/ManageLanguages.php (Line 397)

This apparently happens when looking at the pre-install-screen for language packs (after searching and clicking on "Install", when permissions are checked). The language pack ID is for "German".

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: albertlast on April 12, 2019, 11:27:25 AM

in the admin area you can check which php version you run,
please had mind that you run with you php version 5.4 and pg 9.4 on the lowest spec which is supported by 2.1.
Higher php version 7.0+ and pg 9.5+ would be better.

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: m4z on April 12, 2019, 12:38:53 PM

Quote from: albertlast on April 12, 2019, 11:27:25 AM
in the admin area you can check which php version you run,
please had mind that you run with you php version 5.4 and pg 9.4 on the lowest spec which is supported by 2.1.
Higher php version 7.0+ and pg 9.5+ would be better.

This is what comes with Debian 8. If it really is a version issue, I can try and see if newer prepackaged versions are available for D8 somewhere, but I'd prefer to avoid it.

Quote
Forum version: SMF 2.1 RC2
Current SMF version: SMF 2.1 RC2
GD version: 2.1.1-dev
PostgreSQL engine: PostgreSQL
PostgreSQL version: 9.4.21
PHP: 5.6.40-0+deb8u2
Server version: lighttpd/1.4.35

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: GigaWatt on April 12, 2019, 07:57:15 PM

PHP 5.6 is EOL, so it's not recommended for use any more. 2.1.x supports PHP 7.x out of the box ;).

https://www.php.net/supported-versions.php

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: albertlast on April 13, 2019, 02:11:15 AM

Any way when you upper your version number wouldn't fixed the issue,
so it'S only general advice.

To your problem,
can you try to add this line in front of

Code Select


		// Mark those which are now writable as such.
		foreach ($context['files'] as $type => $data)

Code Select


                print_r($context['files']);
		// Mark those which are now writable as such.
		foreach ($context['files'] as $type => $data)

and give us the html output?

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: m4z on April 13, 2019, 03:24:30 AM

Thanks for your replies.

Quote from: GigaWatt on April 12, 2019, 07:57:15 PM
PHP 5.6 is EOL, so it's not recommended for use any more. [...]

https://www.php.net/supported-versions.php

That, and the fact which version is recommended, seems to be undocumented (https://wiki.simplemachines.org/smf/Requirements_and_recommendations) in the wiki (maybe because 2.1 isn't released yet). The tarballed readme.html (which I only found just now, days after install ::)) says: req: php >= 5.4; rec: php >= 7.0.0.

Quote from: GigaWatt on April 12, 2019, 07:57:15 PM
2.1.x supports PHP 7.x out of the box ;).

Debian 8, which is still LTS (https://wiki.debian.org/DebianReleases#Production_Releases), does not. Even Debian 9, which my hoster doesn't support yet (almost two years after release), only has 7.0 (already EOL), and probably will until 2022.

The print_r output is rather huge (you can easily see which language pack I chose this time):

Code Select

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: albertlast on April 13, 2019, 04:57:55 AM

Well i should mention that you need to switch the view in your browser to source view,
to get the formated view of print_r

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: m4z on April 13, 2019, 05:25:22 AM

Thanks, I don't know much about PHP.

In the meantime I switched my forum from English to German, don't know if that changes the array below (but the error still appears anyway, so it probably doesn't matter).

Code Select

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: GigaWatt on April 14, 2019, 01:34:53 AM

Quote from: m4z on April 13, 2019, 03:24:30 AM
That, and the fact which version is recommended, seems to be undocumented (https://wiki.simplemachines.org/smf/Requirements_and_recommendations) in the wiki (maybe because 2.1 isn't released yet). The tarballed readme.html (which I only found just now, days after install ::)) says: req: php >= 5.4; rec: php >= 7.0.0.

Yes, that's correct ;). PHP 7.x is recommended for 2.1 ;). Actually, since 7.0 is also EOL now, >= 7.1 is recommended now ::) :P.

Quote from: m4z on April 13, 2019, 03:24:30 AM
Debian 8, which is still LTS (https://wiki.debian.org/DebianReleases#Production_Releases), does not. Even Debian 9, which my hoster doesn't support yet (almost two years after release), only has 7.0 (already EOL), and probably will until 2022.

That's an OS support problem, has nothing to do with the fact that some PHP versions are going EOL.

And, let's face it, Debian is not the most recommended Linux OS for a server. Both CentOS 7.6 and 6.10, as well as RHEL 6.10 and 7.6 support PHP 7.3, even though both CentOS 6.10 and RHEL 6.10 are going to be EOL at the end of 2020.

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: Sesquipedalian on April 15, 2019, 02:18:20 AM

The PHP version m4z is using is not relevant to this bug. The problem is a code mistake in ManageLanguages.php.

Tracked in https://github.com/SimpleMachines/SMF2.1/issues/5599

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: Sesquipedalian on April 23, 2019, 01:09:33 PM

Fix has been merged.

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: m4z on April 23, 2019, 03:24:13 PM

Thanks!

Title: Re: ManageLanguages.php (Line 396): Illegal string offset 'writable'
Post by: shawnb61 on February 13, 2020, 10:22:21 PM

The fix for this issue has been merged & is available for download in the latest version over on GitHub.

Simple Machines Community Forum

SMF Development => Bug Reports => Fixed or Bogus Bugs => Topic started by: m4z on April 12, 2019, 04:55:33 AM