Lots of classes do not allow `__debugInfo()` overrides to work
Nessuno ha ancora preso questa issue.
- Lingua principale
- C
- Stelle
- 40.4k
- Fork
- 8.1k
- Merge medio
- 2g 13h
- PR unite (30g)
- 96
Descrizione
Description
Inspired by #11310, I wrote a quick test script to check if there were other classes where creating a subclass with a __debugInfo() method did not work. I found a number of such classes (the one seg fault I filed separately at #16316 since that is a bigger issue than the override just not working).
I tested against b675db4c56dd0de4ea1f5195d587ed90f0096ed8
I did not test against all extensions, `php -m` output provided
[PHP Modules]
bcmath
calendar
Core
ctype
curl
date
dba
dom
enchant
exif
FFI
fileinfo
filter
ftp
gd
gettext
gmp
hash
iconv
intl
json
ldap
libxml
mbstring
mysqli
mysqlnd
odbc
openssl
pcntl
pcre
PDO
pdo_dblib
PDO_Firebird
pdo_mysql
PDO_ODBC
pdo_pgsql
pdo_sqlite
pgsql
Phar
posix
random
readline
Reflection
session
shmop
SimpleXML
snmp
soap
sockets
sodium
SPL
sqlite3
standard
sysvmsg
sysvsem
sysvshm
tidy
tokenizer
xml
xmlreader
xmlwriter
xsl
zend_test
zip
zlib
Approach: create a custom subclass with an empty constructor, and a __debugInfo() method returning ['x' => 'y']. The var_dump() a new instance of that subclass and check the output.
I found 17 classes across the DOM and SimpleXML extension where doing this caused an exception to be thrown relating to invalid state, e.g.
<?php
class Demo extends DOMNode {
public function __construct() {}
public function __debugInfo(): array {
return ['x' => 'y'];
}
}
var_dump( new Demo() );
List of the 17 classes
DOMNode::class,
DOMNameSpaceNode::class,
DOMDocumentFragment::class,
DOMDocument::class,
DOMCharacterData::class,
DOMAttr::class,
DOMElement::class,
DOMText::class,
DOMComment::class,
DOMCdataSection::class,
DOMDocumentType::class,
DOMNotation::class,
DOMEntity::class,
DOMEntityReference::class,
DOMProcessingInstruction::class,
SimpleXMLElement::class,
SimpleXMLIterator::class,
I also found a further 20 classes where the override does not seem to work, across multiple extensions.
List of those classes
array (
0 => 'DatePeriod',
1 => 'DOMImplementation',
2 => 'Dom\\Implementation',
3 => 'DOMNodeList',
4 => 'Dom\\NodeList',
5 => 'DOMNamedNodeMap',
6 => 'Dom\\NamedNodeMap',
7 => 'Dom\\DtdNamedNodeMap',
8 => 'Dom\\HTMLCollection',
9 => 'IntlTimeZone',
10 => 'IntlCalendar',
11 => 'IntlGregorianCalendar',
12 => 'IntlBreakIterator',
13 => 'IntlRuleBasedBreakIterator',
14 => 'IntlCodePointBreakIterator',
15 => 'SplFixedArray',
16 => 'mysqli',
17 => 'mysqli_result',
18 => 'mysqli_stmt',
19 => 'XMLReader',
)
You can see the script I used at https://gist.github.com/DanielEScherzer/c979c15b47bc603758c4c38ba25dd510
PHP Version
dev-master
Operating System
No response
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia con lo script di riproduzione collegato e i relativi controlli con var_dump, quindi esamina le classi DOM, SimpleXML e le altre classi interessate elencate. Riproduci le eccezioni per stato non valido e gli override non funzionanti nelle classi segnalate; il lavoro è completato quando il comportamento di __debugInfo() nelle sottoclassi funziona in modo coerente senza queste eccezioni.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- c, php
- Ambito
- backend
- Tipo di issue
- Bug
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100