<table><tr><td style="">broulik created this revision.<br />broulik added a reviewer: vkrause.<br />Herald added a project: KDE PIM.<br />Herald added a subscriber: kde-pim.<br />broulik requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28921">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>I've noticed it quite often that websites annotate their <tt style="background: #ebebeb; font-size: 13px;"><body></tt> as <tt style="background: #ebebeb; font-size: 13px;">WebPage</tt> and then have a <tt style="background: #ebebeb; font-size: 13px;"><script type="application/ld+json></tt> inside. We would not find such a tag since we're only on the lookout for more microdata once processing one.</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Existing unittests still pass</li>
<li class="remarkup-list-item">Comes with a new one to cover this usecase</li>
</ul>
<ul class="remarkup-list">
<li class="remarkup-list-item">We find <tt style="background: #ebebeb; font-size: 13px;">NewsArticle</tt> annotations in articles on rheinpfalz.de</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R1003 KItinerary: Travel Reservation handling library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28921">https://phabricator.kde.org/D28921</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/structureddata/empty-microdata-with-json-ld-inside.html<br />
autotests/structureddata/empty-microdata-with-json-ld-inside.json<br />
src/generic/structureddataextractor.cpp</div></div></div><br /><div><strong>To: </strong>broulik, vkrause<br /><strong>Cc: </strong>kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>