<table><tr><td style="">emateli marked 2 inline comments as done.<br />emateli 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/D14631">View Revision</a></tr></table><br /><div><div><p>Something that might need a bit of input is <tt style="background: #ebebeb; font-size: 13px;">BatchRenameJob</tt>.</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">This was added a while back so that Dolphin doesn't do a batch rename as a series of single file renames.</li>
<li class="remarkup-list-item">This job takes care of replacing the <tt style="background: #ebebeb; font-size: 13px;">#</tt> placeholder as well.</li>
<li class="remarkup-list-item">What this patch really wants is an API to rename a list of files, into some new names without doing any additional handling except moving the files (as the placeholder replacement is done elsewhere).</li>
<li class="remarkup-list-item">By consulting <a href="https://lxr.kde.org/search?_filestring=&_string=BatchRenameJob" class="remarkup-link" target="_blank" rel="noreferrer">https://lxr.kde.org/search?_filestring=&_string=BatchRenameJob</a> we can see that Dolphin is the only application to make use of this <tt style="background: #ebebeb; font-size: 13px;">BatchRenameJob</tt> API.</li>
<li class="remarkup-list-item">Leaving credit to the original author, since the <tt style="background: #ebebeb; font-size: 13px;">#</tt> placeholder will not be used anymore. Should we re-purpose this class to accept two lists, one with the current names, the other with the new ones and remove the original placeholder implementation (as it would be mostly dead code at this point) while keeping everything else intact.</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14631">https://phabricator.kde.org/D14631</a></div></div><br /><div><strong>To: </strong>emateli, Frameworks, dfaure, mlaurent<br /><strong>Cc: </strong>mlaurent, asensi, rkflx, dfaure, aacid, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>