Skip to content

Gh21768 dom #21773

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
devnexen merged 7 commits into from
Apr 18, 2026
Merged

Gh21768 dom #21773

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6fc440b
build/gen_stub: support asymmetric visibility modifiers.
devnexen Apr 16, 2026
c8155e9
ext/dom: use private(set) instead of @readonly for virtual properties.
devnexen Apr 16, 2026
5bb2eec
Emit asymmetric visibility modifiers in field synopsis.
devnexen Apr 17, 2026
3326690
Emit asymmetric visibility error in dom_write_property.
devnexen Apr 17, 2026
50bfa3a
fix tests
devnexen Apr 17, 2026
8e29897
feedback
devnexen Apr 18, 2026
7e46550
revert back
devnexen Apr 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 11 additions & 1 deletion build/gen_stub.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2354,7 +2354,17 @@ protected function getFlagsByPhpVersion(): VersionFlags
$flags = "ZEND_ACC_PRIVATE";
}

return new VersionFlags([$flags]);
$versionFlags = new VersionFlags([$flags]);

if ($this->flags & Modifiers::PUBLIC_SET) {
$versionFlags->addForVersionsAbove("ZEND_ACC_PUBLIC_SET", PHP_84_VERSION_ID);
} elseif ($this->flags & Modifiers::PROTECTED_SET) {
$versionFlags->addForVersionsAbove("ZEND_ACC_PROTECTED_SET", PHP_84_VERSION_ID);
} elseif ($this->flags & Modifiers::PRIVATE_SET) {
$versionFlags->addForVersionsAbove("ZEND_ACC_PRIVATE_SET", PHP_84_VERSION_ID);
}
Comment thread
kocsismate marked this conversation as resolved.

return $versionFlags;
}

protected function getTypeCode(string $variableLikeName, string &$code): string
Expand Down
Loading
Loading