<table><tr><td style="">eduisters updated this revision to Diff 49386.<br />eduisters edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-prw6kon2l2axdvs/">(Show Details)</a><br />eduisters edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-l4pohv3gkxx5mb7/">(Show Details)</a><br />eduisters 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/D18212">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Properly handle the situation where no storage locations have been configured</li>
<li class="remarkup-list-item">Handle preference changes</li>
<li class="remarkup-list-item">Allow deleting of the last storage location</li>
<li class="remarkup-list-item">Always create the right FileSystemView depending on Build.VERSION</li>
<li class="remarkup-list-item">Remove erroneous leftover from re-fractoring SftpSettingsActivity into SftpSettingsFragment</li>
<li class="remarkup-list-item">Commented out entries needed to debug org.apache.sshd</li>
<li class="remarkup-list-item">Remove unused variables</li>
<li class="remarkup-list-item">Remove optional WRITE_EXTERNAL_STORAGE permission because it is always granted on Build.VERSION < KITKAT</li>
<li class="remarkup-list-item">Remove required WRITE_EXTERNAL_STORAGE permission because it is always granted on Build.VERSION < 23 at installation time and we now use SaF starting Build.VERSION >= 21</li>
<li class="remarkup-list-item">Remove unneeded PreferenceFragmentCompat.OnPreferenceStartScreenCallback</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><div style="padding: 8px 0;">...</div>|API 21+|API 19-|Edit|<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Install this patch and D18211 and verify that the sftp plugin works as expected and that the sdcard is now writable for all android versions except Kitkat (When permissions are granted/storage locations have been configured)<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);"><br />
<br />
<br />
<br />
</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D18212?vs=49345&id=49386">https://phabricator.kde.org/D18212?vs=49345&id=49386</a></div></div><br /><div><strong>BRANCH</strong><div><div>sftp_saf</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18212">https://phabricator.kde.org/D18212</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>build.gradle<br />
res/drawable/ic_arrow_drop_down_24px.xml<br />
res/drawable/ic_delete.xml<br />
res/layout/fragment_storage_preference_dialog.xml<br />
res/layout/preference_checkbox.xml<br />
res/menu/sftp_settings_action_mode.xml<br />
res/values/strings.xml<br />
res/values/styles.xml<br />
res/xml/sftpplugin_preferences.xml<br />
src/org/kde/kdeconnect/Helpers/StorageHelper.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/AndroidFileSystemFactory.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/AndroidFileSystemView.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/AndroidSafFileSystemView.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/AndroidSafSshFile.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/AndroidSshFile.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/RootFile.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/SftpPlugin.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/SftpSettingsFragment.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/SimpleSftpServer.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/StoragePreference.java<br />
src/org/kde/kdeconnect/Plugins/SftpPlugin/StoragePreferenceDialogFragment.java<br />
src/org/kde/kdeconnect/UserInterface/DeviceSettingsActivity.java<br />
src/org/kde/kdeconnect/UserInterface/DeviceSettingsFragment.java</div></div></div><br /><div><strong>To: </strong>eduisters, KDE Connect<br /><strong>Cc: </strong>kdeconnect, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, mikesomov, tctara, apol<br /></div>