Project

General

Profile

Σφάλματα #130

πρόβλημα με $_SESSION['status’]

Added by Panayotis Katsaloulis almost 11 years ago. Updated over 4 years ago.

Status:
Κλειστό
Priority:
Κανονική
Assignee:
-
Target version:
Start date:
11/02/2014
Due date:
% Done:

90%

Estimated time:

Description

στο αρχείο main/lessons.php
στο function getUserLessonInfo
υπάρχει μια σειρά από if’s

όμως, δεν καλύπτονται όλες οι περιπτώσεις (π.χ. σαν admin) με αποτέλεσμα να μη παράγεται το query αφού δεν είναι σωστό το $session->status

Για την ακρίβεια, το $session->status είναι array και έχει τιμή
( [TMA100] => 1 )
(δηλαδή ως array τα μαθήματα) που είναι πολύ περίεργο να έχει πάρει τιμή το session κάτι τέτοιο.

κοιτάζοντας προς τα πίσω βλέπω πως ομοίως και το $_SESSION['status’] έχει την ίδια τιμή, άρα δεν έχει μπει καν σωστά στο session το status.

Δεν μπορώ να καταλάβω πιο βαθιά, αλλά διαπιστώνω 2 θέματα:

1) στο lessons θα πρέπει να υπάρχει και ένα else γενικό, σε περίπτωση που ξεφύγει πάλι κάτι
2) να δούμε γιατί στο session αποθηκεύεται ένα array αντί για τιμή

Associated revisions

Revision 6707:ec582c95e483 (diff)
Added by Thanos Kyritsis almost 11 years ago

proper session update after create course (refs #130)

Revision 15029:ec8090a5f05f (diff)
Added by Thanos Kyritsis almost 11 years ago

proper session update after create course (refs #130)

History

#1

Updated by Thanos Kyritsis almost 11 years ago

  • Status changed from Νέα to Σχόλια
  • Target version set to 3.0
  • % Done changed from 0 to 90

Μόλις έστειλα ένα σχετικό patch. Ο ένοχος ήταν το create course, δεν ενημέρωνε σωστά τα session course statuses. Ενημέρωνε/χάλαγε το session status και γι' αυτό έβγαιναν τα σκασίματα μετά.

Επί της ευκαιρίας, ψιλοδιόρθωσα/refactor και τον κοντινό σε αυτά όλα κώδικα. Αν φαίνεται ΟΚ, ας κλείσουμε το issue.

#2

Updated by Yannis Exidaridis over 4 years ago

  • Status changed from Σχόλια to Κλειστό

Also available in: Atom PDF