Skip to content

Commit 3b24863

Browse files
Code Modernization: Remove xml_set_object() in MagpieRSS::__construct().
The XML Parser extension still supports a quite dated mechanism for method based callbacks, where the object is first set via `xml_set_object()` and the callbacks are then set by passing only the name of the method to the relevant parameters on any of the `xml_set_*_handler()` functions. {{{ xml_set_object( $parser, $my_obj ); xml_set_character_data_handler( $parser, 'method_name_on_my_obj' ); }}} Passing proper callables to the `xml_set_*_handler()` functions has been supported for the longest time and is cross-version compatible. So the above code is 100% equivalent to: {{{ xml_set_character_data_handler( $parser, [$my_obj, 'method_name_on_my_obj'] ); }}} The mechanism of setting the callbacks with `xml_set_object()` has now been deprecated as of PHP 8.4, in favour of passing proper callables to the `xml_set_*_handler()` functions. This is also means that calling the `xml_set_object()` function is deprecated as well. This commit fixes this deprecation for the `MagpieRSS::__construct()` method. The change has not been not covered by tests. This class has been deprecated since WP 3.0.0 and is not covered by tests at all. Adding those now seems superfluous, all the more as the principle of the fix is no different than for the other files, so we can be sure it works anyway. Note: Though this is "officially" an external library, this package is no longer externally maintained. The code style of the fix in the source file is in line with the existing code style for the file. Refs: * https://wiki.php.net/rfc/deprecations_php_8_4#xml_set_object_and_xml_set_handler_with_string_method_names * https://www.php.net/manual/en/function.xml-set-object.php * https://www.php.net/manual/en/ref.xml.php Follow-up to [4399]. Props jrf. See #62061. git-svn-id: https://develop.svn.wordpress.org/trunk@59063 602fd350-edb4-49c9-b593-d223f7449a82
1 parent ce57101 commit 3b24863

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

src/wp-includes/rss.php

+2-3
Original file line numberDiff line numberDiff line change
@@ -74,11 +74,10 @@ function __construct( $source ) {
7474
# pass in parser, and a reference to this object
7575
# set up handlers
7676
#
77-
xml_set_object( $this->parser, $this );
7877
xml_set_element_handler($this->parser,
79-
'feed_start_element', 'feed_end_element' );
78+
array( $this, 'feed_start_element' ), array( $this, 'feed_end_element' ) );
8079

81-
xml_set_character_data_handler( $this->parser, 'feed_cdata' );
80+
xml_set_character_data_handler( $this->parser, array( $this, 'feed_cdata' ) );
8281

8382
$status = xml_parse( $this->parser, $source );
8483

0 commit comments

Comments
 (0)