Doc #80494 [Com]: DOMElement::remove not documented
Edit report at https://bugs.php.net/bug.php?id=80494&edit=1
ID: 80494
Comment by: contact at perryrylance dot com
Reported by: contact at perryrylance dot com
Summary: DOMElement::remove not documented
Status: Open
Type: Documentation Problem
Package: Documentation problem
Operating System: Windows 10
PHP Version: 8.0.0
Block user comment: N
Private report: N
New Comment:
Hi Benjamin
I see that more methods have been added to DOMElement, which is great.
I'd like to bring my library up to date, drop support for PHP < 7.0.0 and add return types
and matching function signatures to your implementation.
I do have one request - could I perhaps ask that the functions remove, before, after and friends
could possibly return $this?
My library facilitates method chaining, changing some of these functions to return void could break
older projects.
If this is at all possible I'd massively appreciate it.
Thank you.
Previous Comments:
------------------------------------------------------------------------
[2021-02-03 06:03:19] janejenkins19844 at gmail dot com
I will be interested in more similar topics. i see you got really very useful topics , i will be
always checking your blog thanks.
https://www.navyarmyccu.ltd/
------------------------------------------------------------------------
[2020-12-08 11:43:07] contact at perryrylance dot com
Wonderful, thank you so much!
Looks like I can simply remove this from my implementation :)
Thanks again!
------------------------------------------------------------------------
[2020-12-08 11:30:44] beberlei@php.net
Sorry, I am having a bad day, the problem is not the parameters, but that the PHP 8 api has
"void" as return type, so my last attempt, this time its right :-)
public function remove(): void;
As a side-note, PHP 8 now contains so called stubs, that show the right signature directly in PHP
code. See the DOM Extension and remove here: https://github.com/php/php-src/blob/master/ext/dom/php_dom.stub.php#L231
------------------------------------------------------------------------
[2020-12-08 11:28:26] beberlei@php.net
I am sorry, i meant:
public function remove();
------------------------------------------------------------------------
[2020-12-08 11:28:00] beberlei@php.net
Right, this was part of the DOM Living Standard RFC https://wiki.php.net/rfc/dom_living_standard_api
I haven't gotten around to the documentation part yet :-( The signature is without paraemters:
public function delete();
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=80494
--
Edit this bug report at https://bugs.php.net/bug.php?id=80494&edit=1
Thread (9 messages)