niklas.ortmann/webprogrammierung
0
0
Fork 0
Code Issues 2 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

WIP: Anpassung der results seite und erstellung der script-datei #37

Closed
viratex wants to merge 1 commits from SuchergebnisseJS into dev
pull from: SuchergebnisseJS
merge into: niklas.ortmann:dev
Conversation 6 Commits 1 Files Changed -
viratex commented 2026-06-17 14:51:40 +02:00
Collaborator
Copy Link
Copy Source

Die Sortierung liefert nun ein Live-Update im Browser

Die Sortierung liefert nun ein Live-Update im Browser
viratex added 4 commits 2026-06-17 14:51:40 +02:00
Anpassung der results seite und erstellung der script-datei 66eeac372c
Debugging 1 cac8f3046d
Debugging 2 9353a7eaaa
aufraeumen beeab0ec90
niklas.ortmann requested changes 2026-06-17 15:19:50 +02:00
Dismissed
niklas.ortmann left a comment
Owner
Copy Link
Copy Source

Das ist ein toller Ansatz, um nun zukünftig clientseitig die Ergebnisse zu sortieren.

Ich hoffe, dass folgendes klar ist:

php wird auf dem Server ausgeführt und js beim Client.
Daher könnte man überlegen, wie man den Server entlastet, indem man einige Aufgaben clientseitig mit js abgibt.

Das Beziehen der Suchergebnisse muss logischerweise aus dem ArticleManager via php erfolgen. Aber wenn der Nutzer das article-Array schon hat und der Nutzer nur die Sortierung ändern will, dann muss ja nicht noch eine weitere Anfrage an den Server gesendet werden (der ArticleManager soll nicht sortieren!!!). Die Soriterung kann clientseitig übernommen werden - mit js.

Außerdem fehlt das Filtern nach Kategorien immer noch. Dafür kannst du aber auch später einen neuen Branch erstellen.

Das ist ein toller Ansatz, um nun zukünftig clientseitig die Ergebnisse zu sortieren. Ich hoffe, dass folgendes klar ist: php wird auf dem Server ausgeführt und js beim Client. Daher könnte man überlegen, wie man den Server entlastet, indem man einige Aufgaben clientseitig mit js abgibt. Das Beziehen der Suchergebnisse muss logischerweise aus dem ArticleManager via php erfolgen. Aber wenn der Nutzer das article-Array schon hat und der Nutzer nur die Sortierung ändern will, dann muss ja nicht noch eine weitere Anfrage an den Server gesendet werden (der ArticleManager soll nicht sortieren!!!). Die Soriterung kann clientseitig übernommen werden - mit js. Außerdem fehlt das Filtern nach Kategorien immer noch. Dafür kannst du aber auch später einen neuen Branch erstellen.
content/search-results.php Outdated
@@ -158,1 +151,4 @@
</div>
<script src="js/paginator.js"></script>
niklas.ortmann commented 2026-06-17 15:08:37 +02:00
Owner
Copy Link
Copy Source

Das wird im Head in der index.php eingebunden. Orientiere dich bitte an dem dev.

Das wird im Head in der index.php eingebunden. Orientiere dich bitte an dem dev.
content/search-results.php Outdated
@@ -159,0 +153,4 @@
<script src="js/paginator.js"></script>
<script src="js/sorter.js"></script>
niklas.ortmann commented 2026-06-17 15:08:47 +02:00
Owner
Copy Link
Copy Source

s. oben

s. oben
js/sorter.js Outdated
@@ -0,0 +1,26 @@
function initSorter() {
niklas.ortmann commented 2026-06-17 15:11:44 +02:00
Owner
Copy Link
Copy Source

Diese js sorgt nur dafür, dass das Formular halt statt über die Form jetzt über js abgeschickt wird. Wo ist hier denn jetzt der Vorteil der js?

Wenn der Nutzer einen Radio-Button ändert, dann kann diese js das abfangen und statt wieder eine Anfrage an den Server (ArticleManager) zu schicken, kann die js doch selbst die Sortierung vornehmen.

Diese js sorgt nur dafür, dass das Formular halt statt über die Form jetzt über js abgeschickt wird. Wo ist hier denn jetzt der Vorteil der js? Wenn der Nutzer einen Radio-Button ändert, dann kann diese js das abfangen und statt wieder eine Anfrage an den Server (ArticleManager) zu schicken, kann die js doch selbst die Sortierung vornehmen.
viratex added 1 commit 2026-06-17 15:31:09 +02:00
verschieben der script aufrufe in die index.php 3e453e22ec
viratex added 1 commit 2026-06-17 15:39:06 +02:00
updater sorter.js fuer client seite a8df9590fd
niklas.ortmann marked the pull request as work in progress 2026-06-17 16:37:32 +02:00
viratex added 1 commit 2026-06-17 17:48:40 +02:00
Merge branch 'dev' into SuchergebnisseJS e1102eb7db
viratex added 1 commit 2026-06-17 17:58:43 +02:00
sortieren nach likes nun moeglich de4d2fe5c0
viratex added 1 commit 2026-06-17 19:19:34 +02:00
Filter funktion implementiert 358247a2a1
viratex added 1 commit 2026-06-17 19:22:07 +02:00
Debugging 3 48f2e90e49
viratex added 1 commit 2026-06-17 19:30:15 +02:00
Debugging 4 d9eca06197
viratex added 1 commit 2026-06-17 19:33:12 +02:00
Debugging 5 969adc4836
viratex added 1 commit 2026-06-17 19:35:05 +02:00
Debugging 6 520298bae4
viratex added 1 commit 2026-06-17 19:38:04 +02:00
Debugging 7 5c0e978c93
viratex added 1 commit 2026-06-17 19:47:38 +02:00
Anpassung der filter.js 0805958033 
es gab ein problem bei der sortierung, da nur 10 ergebnisse angezeigt werden. mit dieser anpassung koennen ergebnisse 'nachruecken'
viratex added 1 commit 2026-06-17 19:54:35 +02:00
Debugging 8 b3c74c2172
viratex added 1 commit 2026-06-17 20:14:10 +02:00
Implementierung des validators 1e6ac0fcd9
viratex added 1 commit 2026-06-17 20:16:04 +02:00
Revert "Implementierung des validators" 1db77346d0 
This reverts commit 1e6ac0fcd9.
viratex added 1 commit 2026-06-17 20:30:47 +02:00
Implementation des validators und korrektes einbinden 87e7be1c10
viratex added 1 commit 2026-06-17 20:38:22 +02:00
anpassung der error message 057639be7c
viratex added 1 commit 2026-06-17 20:42:34 +02:00
aufraeumen 2 12ad9759cf
viratex marked the pull request as ready for review 2026-06-17 20:43:54 +02:00
viratex commented 2026-06-17 20:45:14 +02:00
Author
Collaborator
Copy Link
Copy Source

Einige AAenderungen:

  1. Suchen laeuft nun auf client seite
  2. Man kann nach Kategorien per dropdown menu filtern
  3. search-validator implementiert, der unzulasessige eingaben, wie script injections, nicht zulaesst
Einige AAenderungen: 1. Suchen laeuft nun auf client seite 2. Man kann nach Kategorien per dropdown menu filtern 3. search-validator implementiert, der unzulasessige eingaben, wie script injections, nicht zulaesst
niklas.ortmann requested changes 2026-06-17 21:27:54 +02:00
Dismissed
niklas.ortmann left a comment
Owner
Copy Link
Copy Source

Bitte pull dir den Branch "Bewertungssystem".

Sonst scheint es gut!

Bitte pull dir den Branch "Bewertungssystem". Sonst scheint es gut!
👍 1
viratex added 1 commit 2026-06-17 21:49:03 +02:00
Merge branch 'dev' into SuchergebnisseJS c117c7e641
viratex added 1 commit 2026-06-17 21:59:17 +02:00
anpassung, damit die like sortierung client seitig ist 395041fd44
niklas.ortmann marked the pull request as work in progress 2026-06-17 22:01:43 +02:00
viratex added 1 commit 2026-06-17 22:03:54 +02:00
anpassung der search results fuer die filterung nach likes 17034c928b
niklas.ortmann requested changes 2026-06-23 14:28:52 +02:00
Dismissed
niklas.ortmann left a comment
Owner
Copy Link
Copy Source

Der Paginator funktioniert durch deine Änderungen nicht mehr!
Bitte prüfen!
Bei 45 Ergebnissen (z.B. bei der Suche von "da") kann man nicht mehr 10, 20, 50, 100 Beiträge zur Ansicht auswählen.

Der Paginator funktioniert durch deine Änderungen nicht mehr! Bitte prüfen! Bei 45 Ergebnissen (z.B. bei der Suche von "da") kann man nicht mehr 10, 20, 50, 100 Beiträge zur Ansicht auswählen.
content/search-results.php Outdated
@@ -120,8 +154,8 @@ $resultCount = count($results);
elseif (isset($_SESSION["search_query"]) && $_SESSION["search_query"] !== "" && $resultCount === 0): ?>
<p>Keine Beiträge zu diesem Suchbegriff gefunden.</p>
<?php
elseif (isset($_SESSION["message"]) && $_SESSION["message"] == "missing_parameters"): ?>
niklas.ortmann commented 2026-06-23 14:26:32 +02:00
Owner
Copy Link
Copy Source

Wird missing_parameters nicht mehr geworfen?

Wird missing_parameters nicht mehr geworfen?
content/search-results.php Outdated
@@ -44,12 +45,11 @@ $resultCount = count($results);
<!-- Links: Seitenleiste für Filter und Suche -->
<aside class="s-res-sidebar">
<!-- Sortierfuntion Box und Such Box-->
niklas.ortmann commented 2026-06-23 14:24:05 +02:00
Owner
Copy Link
Copy Source

Warum wurden die Kommentare entfernt?

Warum wurden die Kommentare entfernt?
content/search-results.php Outdated
@@ -28,3 +28,3 @@
// Nur die Ergebnisse für die aktuelle Seite ausschneiden
$results = array_slice($all_results, $offset, $limit);
//$results = array_slice($all_results, $offset, $limit);
niklas.ortmann commented 2026-06-23 14:23:30 +02:00
Owner
Copy Link
Copy Source

Dadurch machst du den Paginator - glaube ich - kaputt

Dadurch machst du den Paginator - glaube ich - kaputt
viratex added 1 commit 2026-06-30 16:16:19 +02:00
testing cbb662d159
viratex added 1 commit 2026-06-30 21:13:05 +02:00
anpassung der filterJS fuer ergebnislimits 5321534d71
viratex commented 2026-06-30 21:15:51 +02:00
Author
Collaborator
Copy Link
Copy Source

Man kann nun die Anzahl an ergbnissen pro seite auswaehlen und die anzahl der seiten wird entsprechent aktualisiert

Man kann nun die Anzahl an ergbnissen pro seite auswaehlen und die anzahl der seiten wird entsprechent aktualisiert
niklas.ortmann approved these changes 2026-07-01 19:09:43 +02:00
niklas.ortmann left a comment
Owner
Copy Link
Copy Source

Passt.

Passt.
niklas.ortmann added 1 commit 2026-07-01 19:13:33 +02:00
Merge branch 'dev' into SuchergebnisseJS 9fcb09ee05
niklas.ortmann closed this pull request 2026-07-01 19:13:45 +02:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
niklas.ortmann
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
caroline.slt viratex niklas.ortmann
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: niklas.ortmann/webprogrammierung#37
Delete Branch "SuchergebnisseJS"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?