<table><tr><td style="">santilin 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/D22062">View Revision</a></tr></table><br /><div><div><p>This patch adds some patterns to ScriptErrorFilterStrategy to handle some PHP errors so that you can click on the message and the matching file is opened.</p>

<p>Some comments:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Shouldn't the patterns be added by the kdevelop-php plugin</li>
<li class="remarkup-list-item">I needed to know the script working dir in order to handle relative paths. I have hacked it reusing the d->m_buildDir member of OutputModel, but there might be another solution.</li>
<li class="remarkup-list-item">I am not confident that my usage of QDir, QUrl and QFileInfo are the best ones, but the way I have coded it works.</li>
<li class="remarkup-list-item">Probably, the ScriptErrorFilterStrategy::scriptDir member should be moved to a new ScriptErrorFilterStrategyPrivate class to mimic the structure of CompilerErrorFilterStrategy (if needed at all)</li>
<li class="remarkup-list-item">I wonder if testing for empty lines (or shorter than a given lenght) to avoid checking the patterns would be an improvement</li>
<li class="remarkup-list-item">I've set all the items as Informative when there is a match, so I only need to set indicators for error and warnings</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R32 KDevelop</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D22062">https://phabricator.kde.org/D22062</a></div></div><br /><div><strong>To: </strong>santilin<br /><strong>Cc: </strong>kdevelop-devel, hmitonneau, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd<br /></div>