[Bug] File action sheet shows no actions at all for locked files
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 1/5
- 見積もり時間
- 1時間未満
- 初心者へのやさしさ
- 86/100
調査の方向性
FileActionsBottomSheet.handleState()から始めて、displayActions()とdisplayLockInfo()がシートに行を追加する方法を調べます。issueでは、アクションがスキップされる原因となる呼び出し順序を特定し、ロック情報より先にアクションを表示することを提案しています。ロックされたファイルに通常のアクションとロック情報の両方が表示されれば完了です。payloadにはテストファイル名がありません。
索引モデルが issue の本文から書いたものです。
説明
Steps to reproduce
- Lock a file (e.g. open a document in the Text or OnlyOffice app, or use "Lock file" from another client).
- In the Android app, open the folder that contains the locked file.
- Tap the three-dot menu of that file.
Actual behaviour
The bottom sheet shows the file name and a "Locked by the app …" row, but no actions at all — not even edit, rename, download or delete. The sheet is short and cannot be expanded by dragging.
Expected behaviour
The regular action list is shown, plus the lock information row.
Cause
FileActionsBottomSheet.handleState() calls displayLockInfo() before displayActions():
if (state.lockInfo != null) {
displayLockInfo(state.lockInfo)
}
displayActions(state.titleFile, state.actions)
displayActions() starts with if (binding.fileActionsList.isNotEmpty()) return, so as soon as the lock row has been appended, every action is skipped. Only the lock row ends up in the sheet.
Suggested fix
Render the actions first and append the lock info afterwards:
displayActions(state.titleFile, state.actions)
if (state.lockInfo != null) {
displayLockInfo(state.lockInfo)
}
This is also what makes the unlock action reachable — see #17838.
Environment
- Nextcloud Android 35.0.1 (Play) and current master, Android 16, Pixel 9 Pro
- Server: Nextcloud 34 (AIO), files_lock 34.0.1
🤖 Generated with Claude Code
- 主要言語
- Kotlin
- スター
- 5.6k
- フォーク
- 2k
- 平均マージ
- 2日 11時間
- マージ済み PR(30日)
- 97
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
nextcloud/android のほかの issue
-
[Bug] Lock owned by another client (e.g. Text app) cannot be unlocked from the app対応中かも @falkloewert が 2 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
Modification time is not preserved when uploading MediaStore content via Android share対応中かも @alperozturk96 が 27 日前に担当しました。 オープン3. to review bug feature: transfers
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
nextcloud/android#17675 · コメント 4 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
0. Needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
nextcloud/android#17533 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
RC 34.1.0-01: Duplicated wording in creation menu — "New" prefix + creator name produces "New New document" ("Neu Neues Dokument")対応中かも @ChristianTannheimer が 92 日前に担当しました。 オープン0. Needs triage bug
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
0. Needs triage bug
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
メンテナーはふだん 1 日以内に返信
nextcloud/android の issue をすべて見る
似ている issue
-
LiquidGlass / LiquidGlassContainer (iOS): padded content measured at the outer width and overflows by the padding対応中かも @aleclarson が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
NativeScript/NativeScript#11561 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
navikt/mock-oauth2-server#1129 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
-
unconfirmed
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
thunderbird/thunderbird-android#11778 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
software-mansion/enriched-markdown#1042 ·
メンテナーはふだん 2 日以内に返信