<table><tr><td style="">kossebau retitled this revision from "[ftp kio-slave] Fix deletion of directory with non-latin parent path" to "[ftp kio-slave] Fix deletion of directory with non-latin1/utf8 parent path".<br />kossebau edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-h4ilh6rtli5p52v/">(Show Details)</a><br />kossebau edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-o7rdbzqjgzpr7bh/">(Show Details)</a>
</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/D16770">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>type for the Ftp::ftpFolder() arg <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">blocked</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">prevented</span> the compiler from complaining about<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
that coding mismatch.<br />
This principle code issue might have been hidden all the years due to non-latin1<br />
characters being rare with ftp server directory layouts. More, most ftp servers<br />
are using UTF-8 encoding , so the implicit cast via QString::fromUtf8 by<br />
accident was matching what the explicit cast via remoteEncoding()->decode()</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">that coding mism</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">in such cases is doing.<br />
<br />
This p</span>atch<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">. Possibly also non-latin1 characters being rare with</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> makes the code correct and also working for any theoretical ftp</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">ftp </span>server<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"> directory layouts prevented this bug being hit so far</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s which use a non-latin1/utf8 encoding</span>.</div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">No ftp server availble with deletion rights myself</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Tested with am ftp server which uses UTF-8 encoding</span>, <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">so only theoretically</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">deleting folders</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">a bug seen and fixed</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">works as before</span>.</div></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/D16770">https://phabricator.kde.org/D16770</a></div></div><br /><div><strong>To: </strong>kossebau, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>