Skip to content
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

Correctly handle siblings_disallowed from spec #7597

Merged
merged 15 commits into from
Aug 3, 2023
Merged

Correctly handle siblings_disallowed from spec #7597

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
3156ae7
Correctly handle `siblings_disallowed` from spec
swissspidy Aug 3, 2023
b5d6465
Remove debug cruft
swissspidy Aug 3, 2023
f7438c2
Lint fixes
swissspidy Aug 3, 2023
5f14a8a
Use yoda conditions
swissspidy Aug 3, 2023
c191f11
Undo some of the changes
swissspidy Aug 3, 2023
47d384e
Merge branch 'develop' into fix/siblings-disallowed
swissspidy Aug 3, 2023
710c0c4
Undo more changes
swissspidy Aug 3, 2023
b3be692
Use `previousSibling`/`nextSibling`
swissspidy Aug 3, 2023
5dd2342
Lint fixes
swissspidy Aug 3, 2023
891477c
Lint fix
swissspidy Aug 3, 2023
0df4b5d
Apply suggestions from code review
swissspidy Aug 3, 2023
21f8f6f
Fix condition
swissspidy Aug 3, 2023
7b6ce5f
Capture first
swissspidy Aug 3, 2023
588e7ff
Fix condition
swissspidy Aug 3, 2023
7fca058
Remove unnecessary check
swissspidy Aug 3, 2023
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
2 changes: 2 additions & 0 deletions .phpcs.xml.dist
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,8 @@
<element value="DEFAULT_WIDTH"/>
<element value="DEFAULT_HEIGHT"/>
<element value="ampElements"/>
<element value="previousElementSibling"/>
<element value="nextElementSibling"/>
</property>
</properties>
</rule>
Expand Down
3 changes: 3 additions & 0 deletions bin/amphtml-update.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -799,6 +799,9 @@ def GetTagRules(tag_spec):
if tag_spec.HasField('unique_warning'):
tag_rules['unique_warning'] = tag_spec.unique_warning

if tag_spec.HasField('siblings_disallowed'):
tag_rules['siblings_disallowed'] = tag_spec.siblings_disallowed

if tag_spec.HasField('child_tags'):
child_tags = collections.defaultdict( lambda: [] )
for field in tag_spec.child_tags.ListFields():
Expand Down
1 change: 1 addition & 0 deletions bin/latest-extension-versions.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -115,6 +115,7 @@
"amp-sticky-ad" : "1.0",
"amp-story" : "1.0",
"amp-story-360" : "0.1",
"amp-story-audio-sticker" : "0.1",
"amp-story-auto-ads" : "0.1",
"amp-story-auto-analytics" : "0.1",
"amp-story-captions" : "0.1",
Expand Down
165 changes: 101 additions & 64 deletions includes/sanitizers/class-amp-allowed-tags-generated.php
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions includes/sanitizers/class-amp-rule-spec.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ abstract class AMP_Rule_Spec {
const MANDATORY_PARENT = 'mandatory_parent';
const DESCENDANT_TAG_LIST = 'descendant_tag_list';
const CHILD_TAGS = 'child_tags';
const SIBLINGS_DISALLOWED = 'siblings_disallowed';

/*
* HTML Element Attribute rule names
Expand Down
Loading