[PR #1011] [MERGED] Fix ambiguous done column in task sorting #3072

Closed
opened 2026-03-22 14:30:17 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-vikunja/vikunja/pull/1011
Author: @kolaente
Created: 6/24/2025
Status: Merged
Merged: 6/24/2025
Merged by: @kolaente

Base: mainHead: codex/fix-bug-in-vikunja-issue-1009


📝 Commits (1)

  • 7f94543 fix: prefix task fields in sorting

📊 Changes

1 file changed (+5 additions, -4 deletions)

View changed files

📝 pkg/models/task_search.go (+5 -4)

📄 Description

Resolves https://github.com/go-vikunja/vikunja/issues/1009

Summary

  • prefix all task fields with the tasks table when constructing SQL ORDER BY clauses
  • ensure bucket sorting uses the proper table alias

Testing

  • mage lint:fix
  • mage test:feature

https://chatgpt.com/codex/tasks/task_e_685aff1610cc83228e66a9646309697d


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-vikunja/vikunja/pull/1011 **Author:** [@kolaente](https://github.com/kolaente) **Created:** 6/24/2025 **Status:** ✅ Merged **Merged:** 6/24/2025 **Merged by:** [@kolaente](https://github.com/kolaente) **Base:** `main` ← **Head:** `codex/fix-bug-in-vikunja-issue-1009` --- ### 📝 Commits (1) - [`7f94543`](https://github.com/go-vikunja/vikunja/commit/7f94543b194934e3093563c05a1ab8cb56097256) fix: prefix task fields in sorting ### 📊 Changes **1 file changed** (+5 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `pkg/models/task_search.go` (+5 -4) </details> ### 📄 Description Resolves https://github.com/go-vikunja/vikunja/issues/1009 ## Summary - prefix all task fields with the tasks table when constructing SQL ORDER BY clauses - ensure bucket sorting uses the proper table alias ## Testing - `mage lint:fix` - `mage test:feature` ------ https://chatgpt.com/codex/tasks/task_e_685aff1610cc83228e66a9646309697d --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-03-22 14:30:17 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vikunja#3072