<table><tr><td style="">davidllewellynjones updated this revision to Diff 70041.<br />davidllewellynjones edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-ga5xzxblaafzpwf/">(Show Details)</a><br />davidllewellynjones added a comment.
</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/D25256">View Revision</a></tr></table><br /><div><div>

<ol class="remarkup-list">
<li class="remarkup-list-item">Updating <a href="https://phabricator.kde.org/D25256" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D25256</a>: Relax TextPFRun validation to allow LibreOffice PPT import #</li>
<li class="remarkup-list-item">Enter a brief description of the changes included in this update.</li>
<li class="remarkup-list-item">The first line is used as subject, next lines as comment. #</li>
<li class="remarkup-list-item">If you intended to create a new revision, use:</li>
<li class="remarkup-list-item">$ arc diff --create</li>
</ol>

<p>Remove excess spaces from commit message.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">This change allows non-conforming LibreOffice PPT files to be imported.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span><br />
An apparent bug in the LibreOffice PPT exporter makes it output files<div style="padding: 8px 0;">...</div>Calligra's validation allows the files to be imported successfully.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span><br />
In more detail, when loading a drawing each text paragraph in the<div style="padding: 8px 0;">...</div>the sequence.<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span><br />
According to Section 2.9.45 of the PPT specification version 6 [1], the<br />
following fields must be masked out:<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span><br />
masks.leftMargin<div style="padding: 8px 0;">...</div>masks.tabStops<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span><br />
In spite of this LibreOffice includes the leftMargin and indent fields<div style="padding: 8px 0;">...</div>(which is correct).<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">    </span><br />
This patch loosens Calligra's validation to allow these flags to be set.<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R8 Calligra</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D25256?vs=70040&id=70041">https://phabricator.kde.org/D25256?vs=70040&id=70041</a></div></div><br /><div><strong>BRANCH</strong><div><div>allow-libreoffice-ppt-validation (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D25256">https://phabricator.kde.org/D25256</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>filters/libmso/generated/mso.jar<br />
filters/libmso/generated/simpleParser.cpp<br />
filters/libmso/generated/simpleParser.h</div></div></div><br /><div><strong>To: </strong>davidllewellynjones, pvuorela, dcaliste<br /><strong>Cc: </strong>Calligra-Devel-list, davidllewellynjones, dcaliste, ognarb, cochise, vandenoever<br /></div>