<table><tr><td style="">barchiesi updated this revision to Diff 57143.<br />barchiesi retitled this revision from "Fix includeTeamDriveItems not being applied to single File fetch" to "FileFetchJob: unconditionally set `includeTeamDriveItems`, `includeItemsFromAllDrives`, `supportsTeamDrives`, `supportsAllDrives` query params".<br />barchiesi edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-bknkpcxg2wswqmd/">(Show Details)</a><br />barchiesi 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/D20789">View Revision</a></tr></table><br /><div><div><p>It seems like Google updated the v2 and v3 Drive references. The Teamdrive part of the API that I recently added is now considered deprecated and Drives should be used instead. Sintactically they are very similar, if not same apart from the added features.</p>

<p>The deprecation notice you are referring to was also just recently added, while the default value if unspecified remains <tt style="background: #ebebeb; font-size: 13px;">false</tt> (?!). So we need to unconditionally set all 4 to <tt style="background: #ebebeb; font-size: 13px;">true</tt> (<tt style="background: #ebebeb; font-size: 13px;">includeTeamDriveItems</tt>, <tt style="background: #ebebeb; font-size: 13px;">includeItemsFromAllDrives</tt>, <tt style="background: #ebebeb; font-size: 13px;">supportsTeamDrives</tt>, <tt style="background: #ebebeb; font-size: 13px;">supportsAllDrives</tt>), at least until June 1, 2020 when I presume the default will be changed to <tt style="background: #ebebeb; font-size: 13px;">true</tt>.</p></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">The <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">//includeTeamDriveItems// and //</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">`</span>supportsTeamDrives<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">//</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">`</span> flags where not getting applied to a FileFetchJob with a specific file to fetch.  This patch fixes the bug.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R477 KGAPI Library</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D20789?vs=56899&id=57143">https://phabricator.kde.org/D20789?vs=56899&id=57143</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20789">https://phabricator.kde.org/D20789</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>examples/teamdrive/mainwindow.cpp<br />
examples/teamdrive/mainwindow.h<br />
examples/teamdrive/ui/main.ui<br />
src/drive/filefetchjob.cpp<br />
src/drive/filefetchjob.h</div></div></div><br /><div><strong>To: </strong>barchiesi, dvratil<br /><strong>Cc: </strong>kde-pim, LibKGAPI, barchiesi, gennad, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>