Replica add-task silently drops due and wait dates (DateTime.toString is not RFC3339)
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 78/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Ambito
- backend-api-design, mobile-dev
Direzione di ricerca
Start with AddTaskBottomSheet.onSaveButtonClickedForReplica in lib/app/modules/home/views/add_task_bottom_sheet_new.dart and Replica.addTaskToReplica in lib/app/v3/champion/replica.dart, then inspect parse_datetime in rust/src/api.rs. Run the relevant Dart and Rust tests available in the repository; done means replica-created tasks retain the selected due and wait timestamps in RFC3339-compatible form.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Description
On the TaskChampion / replica path, creating a task with a due or wait date from the Add Task sheet reports success, but the date is not stored.
The add sheet puts a Dart DateTime into the replica map. Replica.addTaskToReplica then calls .toString() on that value. Dart’s DateTime.toString() uses a space (2026-10-03 12:00:00.000Z). The Rust FFI parse_datetime only accepts RFC3339 (2026-10-03T12:00:00.000Z). Parse failure becomes None, and set_due / set_wait clear the field. The write still succeeds, so the UI shows “task added”.
The edit path already formats with toIso8601String() (which includes T), so dates set on the detail screen survive. Only create is wrong.
This is source-confirmed on main (90700d4). I could not run the Flutter app in this environment (no Flutter SDK).
Steps to reproduce
- Enable TaskChampion replica mode (
settings_taskr_repl). - Open Add Task.
- Enter a description and pick a Due (and/or Wait) date/time.
- Save. Confirm the success snackbar.
- Open the new task (or list it after refresh).
Expected behavior
The due/wait timestamp chosen in the add sheet is stored on the replica task, same as when the same date is set later on the edit screen.
Actual behavior
The task is created without due / wait. The add sheet still reports success.
Technical analysis
Create path
AddTaskBottomSheet.onSaveButtonClickedForReplica (lib/app/modules/home/views/add_task_bottom_sheet_new.dart) passes:
due: getDueDate(homeController.selectedDates)?.toUtc(),
wait: getWaitDate(homeController.selectedDates)?.toUtc(),
Replica.addTaskToReplica (lib/app/v3/champion/replica.dart) stringifies every attribute with:
map[attr] = newTask[attr].toString();
For a UTC DateTime that is 2026-10-03 12:00:00.000Z (space, not T).
FFI
parse_datetime in rust/src/api.rs:
fn parse_datetime(input: &str) -> Option<DateTime<Utc>> {
if input.trim().is_empty() {
return None;
}
input.parse::<DateTime<Utc>>().ok()
}
Empty or unparsable input → None. add_task then does t.set_due(parse_datetime(&value), …) with no error if parse fails. Rust unit fixtures correctly use 2026-09-01T09:00:00Z(withT`), so they never catch this.
Edit path (works)
TaskcDetailsController.saveTask converts with parsed.toUtc().toIso8601String() before modifyTaskInReplica. That string still goes through .toString(), but it is already RFC3339.
Impact
Users on the replica / TaskChampion path (the GSoC v3 default) lose due and wait dates at creation. Overdue highlighting, wait filtering, and sync to other Taskwarrior clients all miss those dates until the user re-enters them on the detail screen. The success snackbar hides the failure.
Possible fix
- In
addTaskToReplica(and anywhere else aDateTimeis put on the map), sendtoUtc().toIso8601String()instead oftoString(). - Optionally make
parse_datetimereject unparsable non-empty strings instead of treating them as “no date”, so a bad format cannot succeed silently. - Add a regression: add a replica task with a due string in Dart
DateTime.toString()form and assert it is stored (or that the write is refused).
Environment
- Repository: CCExtractor/taskwarrior-flutter
- Branch: main (
90700d4f08a58bcaf7e5804352a7bd565ed7ddd8) - App version in pubspec: 3.0.0+3
- Dart SDK constraint:
>=3.0.5 <4.0.0 - TaskChampion FFI:
rust/src/api.rs(taskchampion2.0.3) - Flutter app not run locally (SDK not on PATH)
- Lingua principale
- Dart
- Stelle
- 244
- Fork
- 179
- Merge medio
- 20m
- PR unite (30g)
- 3
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CCExtractor/taskwarrior-flutter
-
Fix misaligned text "Select the directory" in settings tileForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
CCExtractor/taskwarrior-flutter#639 · 2 commenti ·
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
-
Drawer should close automatically when navigating to a new screenForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
bug
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
CCExtractor/taskwarrior-flutter#645 · 1 commento ·
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
CCExtractor/taskwarrior-flutter#643 · 1 commento ·
Tutte le issue di CCExtractor/taskwarrior-flutter
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
DarkFlippers/qUnleashed#240 ·
I maintainer di solito rispondono entro 1 giorno
-
feat: Order macOS versionsAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
quickemu-project/quickgui#328 ·
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
ApplETS/Notre-Dame#1393 ·
-
(fix) Wrong dateAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
Solvro/mobile-topwr#1239 ·
-
Handle `kbd`Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
freeCodeCamp/mobile#1845 ·