Project

General

Profile

Σφάλματα #125

Μαζική δημιουργία μαθημάτων

Added by Panayotis Katsaloulis over 7 years ago. Updated over 7 years ago.

Status:
Νέα
Priority:
Υψηλή
Assignee:
-
Target version:
Start date:
23/12/2013
Due date:
% Done:

80%

Estimated time:

Description

Η διαδικασία της μαζικής δημιουργίας μαθημάτων δεν λειτουγεί
modules/admin/multicourse.php

Από όσο μπόρεσα να δω, έχει να κάνει με το faculte (που μου μοιάζει υπερβολικά γαλλικό για να είναι σωστό).
Η έλλειψη αυτής της παραμέτρου ανακυκλώνεται σε διάφορα σημεία, γι'αυτό το λάθος εμφανίζεται πολλαπλές φορές

Tα μηνύματα λάθους είναι

Notice: Undefined index: faculte in /Users/teras/Works/Development/WebApps/eClass/eclass/modules/admin/multicourse.php on line 36

Notice: Undefined offset: 1 in /Users/teras/Works/Development/WebApps/eClass/eclass/include/main_lib.php on line 809

Notice: Undefined offset: 0 in /Users/teras/Works/Development/WebApps/eClass/eclass/include/main_lib.php on line 808

Warning: Invalid argument supplied for foreach() in /Users/teras/Works/Development/WebApps/eClass/eclass/modules/create_course/functions.php on line 59

Notice: Undefined variable: set_fac in /Users/teras/Works/Development/WebApps/eClass/eclass/modules/create_course/functions.php on line 67


Related issues

Has duplicate Σφάλματα #103: Λάθος στη σελίδα multicourse (ανάγκη για port στην 3.0)Κλειστό21/03/2013

Actions

Associated revisions

Revision 6558:597aaf9f4315 (diff)
Added by Thanos Kyritsis over 7 years ago

bugfixes for multicourse (refs #125)

Revision 14970:52f6cd12e3f0 (diff)
Added by Thanos Kyritsis over 7 years ago

bugfixes for multicourse (refs #125)

Revision 6590:467512acc63a (diff)
Added by Thanos Kyritsis over 7 years ago

bugfix prof query in multicourse creation (refs #125)

Revision 14981:5da240886d6e (diff)
Added by Thanos Kyritsis over 7 years ago

bugfix prof query in multicourse creation (refs #125)

History

#1

Updated by Yannis Exidaridis over 7 years ago

ναι πρέπει το όνομα πεδίου να αλλαξει

#2

Updated by Yury Kherimyan over 7 years ago

Στο αρχείο modules/admin/multicourse.php
1) σειρά 36
$fac = intval($_POST['faculte']); =>
$fac = $_POST['department'];

2) σειρά 58
η create_course καλείται με 8 παραμέτρους, ενώ δέχεται 7 με αποτέλεσμα να δημιουργούνται μαθήματα με κωδικό $_POST['type'] (pre || post || other).
Κοίταξα λίγο την βάση και την συνάρτηση create_course και δεν χρειάζονται κάπου αυτό το όρισμα, οπότε μάλλον:
$cid = create_course('', $lang, $title, $fac, $vis, $prof_name, $_POST['type'], $_POST['password']); =>
$cid = create_course('', $lang, $title, $fac, $vis, $prof_name, $_POST['password']);

3) σειρά 60
η συνάρτηση activate_subsystems δεν υπάρχει, την έψαξα σε όλο το project και δεν βρήκα κάτι (έψαξα και παραλλαγές - activate - subsystems)

4) σειρά 62-63
db_query("INSERT INTO cours_user =>
db_query("INSERT INTO course_user

SET cours_id = $cid, =>
SET course_id = $cid[1],

5) σειρά 277 - 280
θέλει ένα σετ παρενθέσεων έξω από την μεγάλη AND με το όνομα κ επίθετο:
(surname = 'Διαχειριστής' AND givenname = 'Πλατφόρμας') OR (givenname = 'Διαχειριστής' AND surname = 'Πλατφόρμας') =>
((surname = 'Διαχειριστής' AND givenname = 'Πλατφόρμας') OR (givenname = 'Διαχειριστής' AND surname = 'Πλατφόρμας'))

#3

Updated by Yannis Exidaridis over 7 years ago

η συνάρτηση activate_subsystems() έχει πράγματι μετακινηθεί από την 3.0. Πλέον τα υποσυστήματα που είναι ενεργά θα είναι προκαθορισμένα.

#4

Updated by Thanos Kyritsis over 7 years ago

Έχω μόλις στείλει ένα patch που διορθώνει αρκετά το multicourse.php. Πλέον κατα βάση δουλεύει. Δείτε όμως και αν χρειάζονται επιπλέον λεπτομέρειες και να κλείσουμε το issue αν πρέπει.

#5

Updated by Thanos Kyritsis over 7 years ago

  • Priority changed from Κανονική to Υψηλή
  • Target version set to 3.0
  • % Done changed from 0 to 60
#6

Updated by Yury Kherimyan over 7 years ago

Δεν ενσωμάτωσες την 5η αλλαγή που έγραψα παραπάνω, έτσι αν έχουμε έστω έναν χρήστη test test με status 5 (εκπαιδευόμενο) και δώσουμε την γραμμή
Testing Lesson|test test
θα δημιουργηθεί μάθημα με εκπαιδευτή τον test test διότι το query θα είναι:
SELECT id FROM user WHERE status = 1 AND (surname = test AND givenname = test) OR (givenname = test AND surname = test)
αποτέλεσμα: FALSE AND (TRUE) OR TRUE => FALSE OR TRUE => TRUE
ενώ θα έπρεπε να είναι FALSE AND ((TRUE) OR (TRUE)) => FALSE AND TRUE => FALSE
άρα πρέπει η σειρά 275 να γίνει:
WHERE status = 1 AND ($sql)

#7

Updated by Thanos Kyritsis over 7 years ago

  • % Done changed from 60 to 70

OK, το συμπλήρωσα κι αυτό, thanks Yury.

Εξακολουθεί να χρειάζεται μια τελική επιβέβαιωση (με πιο λεπτομερές testing, πχ στο αν δουλεύουν όλες οι επιλογές της φόρμας, language, κλπ) για να κλείσουμε το issue.

#8

Updated by Yannis Exidaridis over 7 years ago

Όταν δημιουργείται μάθημα με αυτόν τον τρόπο δεν λειτουργεί η εγγραφή χρηστών σε ομάδες χρηστών. (δεν ενημερώνεται σωστά ή δεν δημιουργείται ο πίνακας group_properties). Ισχύει στην 2.8 - υποθέτω και στην 3.0

#9

Updated by Yannis Exidaridis over 7 years ago

  • % Done changed from 70 to 80

Πλέον γίνεται το κατάλληλο insert στον πίνακα group_properties οπότε οι αλλαγές ρυθμίσεων στις ομάδων χρηστών δουλεύουν σωστά.

Υπάρχει όμως πρόβλημα. Δεν μπορείς να επιλέξεις σχολή / τμήμα. Ενώ εμφανίζεται το δέντρο η επιλογή δεν δουλεύει σωστά. (επιλέγεται κάθε φορά το πρώτο)

Also available in: Atom PDF