Respons case 6 - Fullstack PHP / MySQL

Här följer respons till caset som gäller modul Fullstack PHP / MySQL. Responsen finns också i form av GitHub Issues - se ditt repo och under fliken "Issues". Under rubriken Kompletteringar framgår eventuella kompletteringar som du måste göra för att få godkänt på caset i den här modulen. Kommentarer i övrigt handlar om din kod som kan vara bra för dig att känna till - men inget som du måste ändra.

Det finns ett par kompletteringar som du måste göra för att bli godkänd - se under rubriken längre ner.

Responsen utgår från de grundläggande krav som finns angivna i caset: https://github.com/Glimakra-Webbutvecklare-2023/Case-PHP-MySQL

En README.md i gitrepot med följande delar:
- Installation: Vad behövs för att man själv ska kunna köra appen?
- Screenshots: minst en screenshot (eller gif) som visar appen under användning
En användare ska kunna registrera ett konto
En användare ska kunna logga in och logga ut
Inloggad användare ska kunna lägga till verksamheter
Verksamheter ska kunna redigeras och raderas av den användare som skapat resursen
Alla verksamheter ska finnas listade för den som besöker applikationen
Alla formulärsfält som registeras till databasen ska valideras (att rätt och rimlig typ att data lagras)

Kommentarer

README.md och installation

I filen README.md saknas det en bild / screenshot.

Screenshots: minst en screenshot (eller gif) som visar appen under användning

Validering av formulär

Bra, du validerar formulär på både frontend och backend!

Kod, funktioner, namngivning, logik etc

Du har gjort en sidan med tilltalande design, din kod är dokumenterad och logiken fungerar väl. Jag tycker också att redovisningen fungerade väl i form av en inspelning. Det har du gjort tidigare, och jag vill uppmuntra dig till att redovisa i samband med det datum som finns redovisning av case i olika moduler.

Kompletteringar

I grundläggande krav: - Alla verksamheter ska finnas listade för den som besöker applikationen

I praktiken innebär det att man ska kunna se vad som är registrerat som resurser - i ditt projekt olika restauranger.

I meny för den om inte är registrerad finns valen:

Home
Register
Login

Via de valen kan man inte se restauranger som finns sparade. Om länken för att upptäcka restauranger fanns med så hade man kunnat se det:

http://localhost:8060/discover.php

Men, då visas en meny som anger att man kan logga ut (tolkar man som att man är inloggad), och att man dessutom kan lägga till innehåll i databasen - vilket man inte ska kunna göra om man inte är inloggad.

Nu kan man som anonym besökare via url:en ... http://localhost:8060/share.php

... lägga till en restaurang anonymt - och det ska man inte kunna göra.

Se till att den som besöker en sida med rättigheter att lägga till / redigera / radera resurs dirigeras tillbaka till din startsida. Exempelvis så här för sidan http://localhost:8060/share.php

<?php
include "_includes/database_connection.php";
session_start();

if (!$_SESSION['user_id']) {
    header("Location: /");
}

// ...

En variant för att nå kravet "Alla verksamheter ska finnas listade för den som besöker applikationen" är att låta anonyma besökare kunna navigera till sidan http://localhost:8060/discover.php

Hade den sidan haft en annan meny så hade den som inte är inloggad kunnat se publika resurser.

Om kompletteringar

Eftersom du redovisade efter vårterminens slut så kommer den här responsen sent. Om du behöver handledning så hör av dig i Teams till mig. Det är inte ngn brådska med att komplettera caset i nuläget. Mitt råd är att du försöker sikta på att komplettera innan du påbörjar examensarbetet under vårterminen.

Bra jobbat med projektet! Jag kan se att du lagt mycket tid på din kod.

Du behöver komplettera enligt ovan för att bli godkänd på modulen.

bypiyi / CASE6-PHP-SQL

Respons #1