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.

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

Reduce instances when u-url is implied #183

Closed
gRegorLove opened this issue Jul 24, 2018 · 0 comments
Closed

Reduce instances when u-url is implied #183

gRegorLove opened this issue Jul 24, 2018 · 0 comments
Assignees
Milestone

Comments

@gRegorLove
Copy link
Member

Per spec update, don't imply u-url if there are other u-* or a child microformat.

Minimized example from an earlier version of http://diggingthedigital.com//Waar-te-beginnen-met-Webmentions/:

<article class="post h-entry">
  <h1 class="p-name"><a href="/Waar-te-beginnen-met-Webmentions/">Waar te beginnen met Webmentions</a></h1>
  <div class="entry e-content">
    <p>Er zijn van die momenten dat ik het liefste de hele dag ga zitten puzzelen hoe ik nu de <a href="/webmentions/">webmentions</a> op deze site in orde moet maken. Het loopt allemaal nog steeds niet lekker, maar ik weet niet goed welke kant ik op moet denken en werken voor een oplossing.</p>
    <p>Waar loop ik nog tegen aan?</p>
    <p>...</p>    
  </div>
  <a href="https://news.indieweb.org/nl" class="u-syndication"></a>
</article>

Currently parses:

"items": [
    {
        "type": [
            "h-entry"
        ],
        "properties": {
            "name": [
                "Waar te beginnen met Webmentions"
            ],
            "syndication": [
                "https://news.indieweb.org/nl"
            ],
            "content": [
                {
                    "html": "<p>Er zijn van die momenten dat ik het liefste de hele dag ga zitten puzzelen hoe ik nu de <a href=\"/webmentions/\">webmentions</a> op deze site in orde moet maken. Het loopt allemaal nog steeds niet lekker, maar ik weet niet goed welke kant ik op moet denken en werken voor een oplossing.</p>\n    <p>Waar loop ik nog tegen aan?</p>\n    <p>...</p>",
                    "value": "Er zijn van die momenten dat ik het liefste de hele dag ga zitten puzzelen hoe ik nu de webmentions op deze site in orde moet maken. Het loopt allemaal nog steeds niet lekker, maar ik weet niet goed welke kant ik op moet denken en werken voor een oplossing. \n    Waar loop ik nog tegen aan? \n    ..."
                }
            ],
            "url": [
                "https://news.indieweb.org/nl"
            ]
        }
    }
]

Expected:

"items": [
    {
        "type": [
            "h-entry"
        ],
        "properties": {
            "name": [
                "Waar te beginnen met Webmentions"
            ],
            "syndication": [
                "https://news.indieweb.org/nl"
            ],
            "content": [
                {
                    "html": "<p>Er zijn van die momenten dat ik het liefste de hele dag ga zitten puzzelen hoe ik nu de <a href=\"/webmentions/\">webmentions</a> op deze site in orde moet maken. Het loopt allemaal nog steeds niet lekker, maar ik weet niet goed welke kant ik op moet denken en werken voor een oplossing.</p>\n    <p>Waar loop ik nog tegen aan?</p>\n    <p>...</p>",
                    "value": "Er zijn van die momenten dat ik het liefste de hele dag ga zitten puzzelen hoe ik nu de webmentions op deze site in orde moet maken. Het loopt allemaal nog steeds niet lekker, maar ik weet niet goed welke kant ik op moet denken en werken voor een oplossing. \n    Waar loop ik nog tegen aan? \n    ..."
                }
            ]
        }
    }
]

Reference: microformats/microformats2-parsing#36

@gRegorLove gRegorLove added this to the 0.4.4 milestone Aug 1, 2018
@aaronpk aaronpk modified the milestones: 0.4.4, 0.4.5 Aug 1, 2018
Zegnat added a commit to Zegnat/php-mf2 that referenced this issue Aug 7, 2018
Fixes microformats#183. Adds a test based on that issue. Fixes an old test that
accidentally tests for the child microformats case.
Zegnat added a commit to Zegnat/php-mf2 that referenced this issue Aug 7, 2018
Fixes microformats#183. Adds a test based on that issue. Fixes an old test that
accidentally tests for the child microformats case.
@Zegnat Zegnat self-assigned this Aug 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants