<table><tr><td style="">albertvaka 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><p>I've tested it on API 19, API 21, and API 28 and seems to work well.</p>
<p>Just a few minor things inlined, plus that field being editable on my OnePlus 6 with API 28.</p>
<p>Iron those tiny things and you have my +1.</p>
<p>Note that I rebased the code onto master! You can get my changes with <tt style="background: #ebebeb; font-size: 13px;">arc patch D18212</tt> instead of rebasing it yourself and re-doing the work.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D18212#inline-109693">View Inline</a><span style="color: #4b4d51; font-weight: bold;">SftpPlugin.java:177</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span style="color: #aa4000">if</span> <span style="color: #aa2211">(</span><span class="n">Build</span><span style="color: #aa2211">.</span><span style="color: #354bb3">VERSION</span><span style="color: #aa2211">.</span><span style="color: #354bb3">SDK_INT</span> <span style="color: #aa2211"><=</span> <span class="n">Build</span><span style="color: #aa2211">.</span><span style="color: #354bb3">VERSION_CODES</span><span style="color: #aa2211">.</span><span style="color: #354bb3">KITKAT</span><span style="color: #aa2211">)</span> <span style="color: #aa2211">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span class="n">addCameraShortcuts</span> <span style="color: #aa2211">=</span> <span class="n">prefs</span><span style="color: #aa2211">.</span><span style="color: #354bb3">getBoolean</span><span style="color: #aa2211">(</span><span class="n">context</span><span style="color: #aa2211">.</span><span style="color: #354bb3">getString</span><span style="color: #aa2211">(</span><span class="n">R</span><span style="color: #aa2211">.</span><span style="color: #354bb3">string</span><span style="color: #aa2211">.</span><span style="color: #354bb3">sftp_preference_key_add_camera_shortcut</span><span style="color: #aa2211">),</span> <span style="color: #000a65">false</span><span style="color: #aa2211">);</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span style="color: #aa2211">}</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">This default should be <tt style="background: #ebebeb; font-size: 13px;">true</tt> to match the default state of the setting. I had to disable and re-enable the setting for the Camera folder to appear.</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D18212#inline-109695">View Inline</a><span style="color: #4b4d51; font-weight: bold;">SftpSettingsFragment.java:322</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span class="n">String</span> <span class="n">displayName</span> <span style="color: #aa2211">=</span> <span style="color: #aa2211">(</span><span class="n">Build</span><span style="color: #aa2211">.</span><span style="color: #354bb3">VERSION</span><span style="color: #aa2211">.</span><span style="color: #354bb3">SDK_INT</span> <span style="color: #aa2211">==</span> <span style="color: #601200">19</span> <span style="color: #aa2211">||</span> <span class="n">info</span><span style="color: #aa2211">.</span><span style="color: #354bb3">readonly</span><span style="color: #aa2211">)</span> <span style="color: #aa2211">?</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"> <span class="n">displayNameReadOnlyBuilder</span><span style="color: #aa2211">.</span><span style="color: #354bb3">toString</span><span style="color: #aa2211">()</span> <span style="color: #aa2211">:</span> <span class="n">displayNameBuilder</span><span style="color: #aa2211">.</span><span style="color: #354bb3">toString</span><span style="color: #aa2211">();</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">I can write to the internal phone storage (sdcard0) on API 19, although it appears as read only because of this condition. Maybe you had a different experience while testing?</p>
<p style="padding: 0; margin: 8px;">To simplify this code, since it's only for old APIs, maybe remove appending the "read only" string? It could be that every device is different, and determining if it is actually going to be read-only is difficult. If that's the case, I don't think it's worth trying to get it right just for old APIs.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R225 KDE Connect - Android application</div></div></div><br /><div><strong>BRANCH</strong><div><div>arcpatch-D18212</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>To: </strong>eduisters, KDE Connect, albertvaka, sredman<br /><strong>Cc: </strong>albertvaka, sredman, kdeconnect, ctakano, vporvaznik, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, domson, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, lemuel, menasshock, mikesomov, tctara, apol<br /></div>