Universal-Omega / DynamicPageList3

GNU General Public License v3.0
18 stars 22 forks source link

"deprecated" errors (Sanatizer.php & Parser.php) #248

Open OfficialCRUGG opened 11 months ago

OfficialCRUGG commented 11 months ago

DynamicPageList3 version: 3.5.1 MediaWiki version: 1.40.0 PHP version: 8.1.23 (apache2handler)

List of steps to reproduce (step by step, including full links if applicable):

What happens?: I get the following output at the top of my page:

Deprecated: trim(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/includes/parser/Sanitizer.php on line 839

Deprecated: trim(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/includes/parser/Sanitizer.php on line 839

Deprecated: trim(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/includes/parser/Sanitizer.php on line 839

Deprecated: trim(): Passing null to parameter #1 ($string) of type string is deprecated in /var/www/html/includes/parser/Sanitizer.php on line 839

Deprecated: str_replace(): Passing null to parameter #2 ($replace) of type array|string is deprecated in /var/www/html/extensions/DynamicPageList3/includes/Parse.php on line 634

Deprecated: str_replace(): Passing null to parameter #2 ($replace) of type array|string is deprecated in /var/www/html/extensions/DynamicPageList3/includes/Parse.php on line 634

What should have happened instead?: The content should render without any problems

Browser information, screenshots and other applicable information: Firefox 120.0.1

image image

OfficialCRUGG commented 11 months ago

I am not sure if this is related, but since it has something to do with rendering/parsing: I have the following in my LocalSettings.php because PortableInfobox doesn't work correctly otherwise:

$wgTidyConfig = [
        'driver' => 'RemexHtml',
        'pwrap' => false,
];
paladox commented 11 months ago

Fixed with https://github.com/Universal-Omega/DynamicPageList3/commit/6f2834a5bca0e3a20dbcec3ffde194df96d16b8a and https://github.com/Universal-Omega/DynamicPageList3/commit/811467f206447e127957d6a8362bf482686394e9

OfficialCRUGG commented 11 months ago

Fixed with https://github.com/Universal-Omega/DynamicPageList3/commit/6f2834a5bca0e3a20dbcec3ffde194df96d16b8a and https://github.com/Universal-Omega/DynamicPageList3/commit/811467f206447e127957d6a8362bf482686394e9

Great! How does your release schedule work? When can we expect this in an official release?

Universal-Omega commented 11 months ago

I will do an official release in the next week or so.