feat: allow editing of packages removed from the database #130
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MartinGauk
requested changes
Aug 29, 2024
There was a problem hiding this comment.
Mein Änderungswunsch rührte ja daher, dass die Fragen noch funktionieren sollen, selbst wenn die Paketinformationen in der DB gelöscht wurden, weil der Server temporärer bei einer Synchronisation keine Pakete zurückgibt. Bis zur nächsten Synchronisation können dann zwar keine Pakete für neue Fragen ausgewählt werden, aber die bestehenden Fragen müssen in jedem Fall weiterhin aufgerufen werden können, gerne auch noch bearbeitet werden können.
| @@ -176,9 +183,9 @@ public function upsert_question(object $question): void { | |||
| // Insert a new record with the question state only containing the options. | |||
| $questionid = $DB->insert_record(self::QUESTION_TABLE, [ | |||
| 'questionid' => $question->id, | |||
| 'feedback' => '', | |||
| // TODO: retrieve the identifier directly from the package? | |||
There was a problem hiding this comment.
Der Identifier wird im Server generiert und könnte auch direkt mitgeliefert werden, falls es mal eine Änderung von @{namespace}\{shortname} geben sollte, müsste es nur dort angepasst werden.
janbritz
changed the title
qtype_questionpy table
MartinGauk
approved these changes
Oct 23, 2024
janbritz
force-pushed
the
feat/update-qtype-questionpy-table
branch
from
56f67e0 to
dff56c5
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Closes #129.
Zusätzlich:
Eine Frage mit einem QPy-Paket, welches zuvor in der Datenbank war und gelöscht wurde, kann immernoch editiert werden. Es kann allerdings nicht mehr favorisiert werden, die Option im Drop-Down-Menü erscheint nicht mehr, und es wird auch nicht im Recently-Used-Tab angezeigt.
Es wird darauf hingewiesen, dass dieses Paket nicht mehr über die Suche zu finden ist (siehe gif).
Gelöschte Pakete werden jetzt auch aus den Favoriten gelöscht.