<table><tr><td style="">kossebau created this revision.<br />kossebau added reviewers: astippich, mgallien, michaelh.<br />Restricted Application added projects: Frameworks, Baloo.<br />Restricted Application added subscribers: Baloo, kde-frameworks-devel.<br />kossebau 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/D13885">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p><a href="https://phabricator.kde.org/R286:7f9de32eedff7f81818145001fc38bbed01de1b7" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">7f9de32eedff7f81818145001fc38bbed01de1b7</a> made extraction of audio<br />
properties depending on the presence of any tag, by moving the extraction<br />
into the "if (!tags->isEmpty())" branch.<br />
This dependency is not necessary and prevents extracting the audio data<br />
for files without a tag (as sideeffect broke a test in baloo-widgets).</p>

<p>Existing sample autotest files in no-meta/ subdir have not catched this,<br />
as they still had an ID3 tag content (TSSE), so tags->isEmpty() was false.</p>

<p>Patch moves the existing files in a new subdir empty-meta/, to reflect they<br />
have some metadata but appear empty for the extraction. And adds a new test<br />
file with no metadata at all (for now only mp3 as I have no idea about<br />
metadata for the others formats)</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>New no-meta/test.mp3 sample file no longer fails with the test.<br />
Unit test extractortest in baloo-widgets also works again, as audio metadata<br />
is extracted again from a file with no id3 data.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R286 KFileMetaData</div></div></div><br /><div><strong>BRANCH</strong><div><div>restoreaudiopropextractionwithouttags</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13885">https://phabricator.kde.org/D13885</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/samplefiles/empty-meta/test.flac<br />
autotests/samplefiles/empty-meta/test.m4a<br />
autotests/samplefiles/empty-meta/test.mp3<br />
autotests/samplefiles/empty-meta/test.mpc<br />
autotests/samplefiles/empty-meta/test.ogg<br />
autotests/samplefiles/empty-meta/test.opus<br />
autotests/samplefiles/no-meta/test.flac<br />
autotests/samplefiles/no-meta/test.m4a<br />
autotests/samplefiles/no-meta/test.mp3<br />
autotests/samplefiles/no-meta/test.mpc<br />
autotests/samplefiles/no-meta/test.ogg<br />
autotests/samplefiles/no-meta/test.opus<br />
autotests/taglibextractortest.cpp<br />
src/extractors/taglibextractor.cpp</div></div></div><br /><div><strong>To: </strong>kossebau, astippich, mgallien, michaelh<br /><strong>Cc: </strong>kde-frameworks-devel, Baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>