Bug 31158 - Refine exam mode
Refine exam mode
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Exam mode
UCS@school 3.1
Other Linux
: P5 normal (vote)
: UCS@school 3.1 R2
Assigned To: Alexander Kläser
Florian Best
: interim-2
: 31246 (view as bug list)
Depends on: 30831
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-24 11:06 CEST by Alexander Kläser
Modified: 2013-06-07 21:39 CEST (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2013-04-24 11:06:17 CEST
The following points have not yet been implemented with Bug 30831:
* Warn if an exam project already exists
* Warn if the chosen computer room is being used
* Read the max upload size from UCRV umc/server/upload/max and configure the MultiUploader accordingly
* JS-side validation of user input
Comment 1 Alexander Kläser univentionstaff 2013-04-24 12:47:04 CEST
More points:
* "Directory name" goes on the second page of the wizard
* Show summary of computer room settings at the end
* Computer room settings in the wizard can be preselected via UCR variables
* A confirmation dialog is missing for finishing/collecting an exam
* "Stop exam" → "Finish exam"
* The order of "Finish exam" and "Collect exam" can be swapped (maybe "Finish exam" with red text color?)
* The progress descriptions for finishing an exam are slightly incorrect
* Room settings should rather be set to their default values after finishing an exam
* Check whether "examDescription" is correctly handed over to SettingsDialog
Comment 2 Alexander Kläser univentionstaff 2013-04-24 16:58:27 CEST
(In reply to comment #1)
> ...
> * The order of "Finish exam" and "Collect exam" can be swapped (maybe "Finish
> exam" with red text color?)

→ maybe an icon would even be better than color?

> ...
> * Check whether "examDescription" is correctly handed over to SettingsDialog

→ I think this can be removed from the SettingsDialog, as the description can be fetched from the project file in the backend
Comment 3 Alexander Kläser univentionstaff 2013-04-25 11:27:36 CEST
More points:
* Translations have not yet been done
* Currently also 0 users can be specified which in turn leads to a float division error
Comment 4 Alexander Kläser univentionstaff 2013-05-06 11:43:26 CEST
*** Bug 31246 has been marked as a duplicate of this bug. ***
Comment 5 Alexander Kläser univentionstaff 2013-05-07 16:45:47 CEST
When starting the exam module for a second exam, the last entries in the file upload lists are visible.
Comment 6 Alexander Kläser univentionstaff 2013-05-08 12:38:05 CEST
The following points have been adjusted in the computer room module:
* A confirmation dialogue has been added for finishing/collecting an exam
* "Stop exam" → "Finish exam"
* Swapped the order of "Finish exam" and "Collect exam"
* Added icons for finishing/collecting an exam
* Removed "examDescription" from SettingsDialog
* Adjusted translations
Comment 7 Alexander Kläser univentionstaff 2013-05-13 17:34:10 CEST
More updates for the exam module:
* Warn if an exam project already exists
* Warn if the chosen computer room is being used (block if room is in exam mode)
* Validation of user input (e.g. >= 1 users), avoid division error (x/0) if groups are empty
* Read the max upload size from UCRV umc/server/upload/max and configure the MultiUploader accordingly
* "Directory name" goes on the second page of the wizard
* Computer room settings in the wizard can be preselected via UCR variables (variables are documented in UCR)
* When starting the exam module a second time, the chosen entries in the file uploader are still visible → Bug 31318
* Adjust progress information for collecting exam ("preparing account" was not correct)

More adjustments for the computer room:
* More specific error handling with information when a room is in use and/or in exam mode
* Error handling when stopping an exam in case no files have been distributed or the project file does not exist anymore
Comment 8 Sönke Schwardt-Krummrich univentionstaff 2013-05-16 17:44:11 CEST
16.05.13 17:29:39.559  MODULE      ( PROCESS ) : Einrichtung des Computerraumes für den Klassenarbeitsmodus... - beendet...
16.05.13 17:29:39.559  MODULE      ( ERROR   ) : Exception during start_exam:   File "/usr/lib/pymodules/python2.6/notifier/threads.py", line 82, in _run
    tmp = self._function()
  File "/usr/lib/pymodules/python2.6/univention/management/console/modules/schoolexam/__init__.py", line 200, in _thread
    entries = [ientry for ientry in [ util.distribution.openRecipients(idn, ldap_user_read, search_base) for idn in request.options.get('recipients', []) ] if ientry ]
  File "/usr/lib/pymodules/python2.6/univention/management/console/modules/distribution/util.py", line 240, in openRecipients
    userobj.open()
  File "/usr/lib/pymodules/python2.6/univention/admin/handlers/users/user.py", line 1499, in open
    self['primaryGroup']=None
  File "/usr/lib/pymodules/python2.6/univention/admin/handlers/__init__.py", line 214, in __setitem__
    raise univention.admin.uexceptions.valueRequired, _('The property %s is required') % self.descriptions[key].short_description

valueRequired: Die Eigenschaft Primäre Gruppe wird benötigt.
Comment 9 Alexander Kläser univentionstaff 2013-05-17 09:45:09 CEST
(In reply to comment #8)
> ...
> valueRequired: Die Eigenschaft Primäre Gruppe wird benötigt.

This happens if for any reason a user is member of a group but the user object has been removed. I observed exam users (which have been removed) being present in a class group. A try except block could be used to mask this problem. However, I am not sure yet what is the source of the problem.
Comment 10 Alexander Kläser univentionstaff 2013-05-17 13:00:33 CEST
(In reply to comment #9)
> (In reply to comment #8)
> > ...
> > valueRequired: Die Eigenschaft Primäre Gruppe wird benötigt.
> 
> This happens if for any reason a user is member of a group but the user object
> has been removed. I observed exam users (which have been removed) being present
> in a class group. A try except block could be used to mask this problem.
> However, I am not sure yet what is the source of the problem.

This problem should be fixed:

ucs-school-umc-distribution (8.0.9-1) unstable; urgency=low
.
 * Bug #31158: make sure that not existing user objects are ignored


The following points have been addressed, as well:
* When stopping an exam, the computer room is reset to its default values
* An exam user account is only removed if it is not in use in a parallel exam (could not be tested extensively due to Bug 31389)
Comment 11 Sönke Schwardt-Krummrich univentionstaff 2013-05-17 13:47:13 CEST
I read all dialogs of the exam module carefully (it took some time) and in the end when I tried to start the exam, I got the following error message:

"Ein unerwarteter Fehler trat währende der Vorbereitung auf: {'desc': "Can't contact LDAP server"}"

Even I press "Weiter" and try it again, it does not work. Same error message.
Comment 12 Sönke Schwardt-Krummrich univentionstaff 2013-05-17 15:47:35 CEST
After creating a new exam session, the UMC module reports:

   Die Vorbereitung der Klassenarbeit war erfolgreich.

but within the logfile the following errors are found (due to bug 31389):

17.05.13 05:09:47.233  MODULE      ( WARN    ) : Could not create exam user account for: uid=tanja516,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local
17.05.13 05:09:47.233  MODULE      ( PROCESS ) : Vorbereitung der Klassenarbeitskonten - Koch, Ulrich (ulrich514)
17.05.13 05:09:47.397  MODULE      ( WARN    ) : Could not create exam user account for: uid=ulrich514,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local

The UMC module should show occurent errors.
Comment 13 Sönke Schwardt-Krummrich univentionstaff 2013-05-17 16:13:25 CEST
* On the exam reboot page, there is no initial explanation why all computers have 
  to be rebooted.
* Which item/widget indicates that a connection to a windows client has bee 
  established successfully?
Comment 14 Alexander Kläser univentionstaff 2013-05-17 18:50:19 CEST
(In reply to comment #11)
> I read all dialogs of the exam module carefully (it took some time) and in the
> end when I tried to start the exam, I got the following error message:
> 
> "Ein unerwarteter Fehler trat währende der Vorbereitung auf: {'desc': "Can't
> contact LDAP server"}"
> 
> Even I press "Weiter" and try it again, it does not work. Same error message.

This is due to an LDAP timeout that happened in the thread function. I moved the first LDAP operation outside the thread such that the LDAP_Connection wrapper can retry a second time.

fixed in → ucs-school-umc-exam (0.1.33-1)


(In reply to comment #12)
> After creating a new exam session, the UMC module reports:
> 
>    Die Vorbereitung der Klassenarbeit war erfolgreich.
> 
> but within the logfile the following errors are found (due to bug 31389):
> 
> 17.05.13 05:09:47.233  MODULE      ( WARN    ) : Could not create exam user
> account for: uid=tanja516,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local
> 17.05.13 05:09:47.233  MODULE      ( PROCESS ) : Vorbereitung der
> Klassenarbeitskonten - Koch, Ulrich (ulrich514)
> 17.05.13 05:09:47.397  MODULE      ( WARN    ) : Could not create exam user
> account for: uid=ulrich514,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local
> 
> The UMC module should show occurent errors.

This should be discussed in Bug 31389.


(In reply to comment #13)
> * On the exam reboot page, there is no initial explanation why all computers
>   have to be rebooted.

Good point, fixed in ucs-school-umc-exam (1.0.1-0).

> * Which item/widget indicates that a connection to a windows client has bee 
>   established successfully?

The icon, but it is certainly not explicit enough, I added a new column in ucs-school-umc-exam (1.0.1-0).

Also added the summary of the exam configuration in ucs-school-umc-exam (1.0.1-0).
Comment 15 Florian Best univentionstaff 2013-05-21 17:13:51 CEST
(In reply to comment #0)
> The following points have not yet been implemented with Bug 30831:
> * Warn if an exam project already exists
→ REOPEN
the error message is OK but the room is then displayed as 'writes an exam' (a new exam really starts)

> * Warn if the chosen computer room is being used
→ OK

> * Read the max upload size from UCRV umc/server/upload/max and configure the
> MultiUploader accordingly
→ OK
REOPEN: if i 1. add files 2. create a exam 3. end exam 4. create new exam: the files from exam1 are in the file list (not removed) 

> * JS-side validation of user input
→ OK
REOPEN: missing red exclamation mark when an invalid value is set

(In reply to comment #1)
> More points:
> * "Directory name" goes on the second page of the wizard
→ OK
→ Is it OK that the name also changes if it was already edited by hand?

> * Show summary of computer room settings at the end
→ OK, looks great now

> * Computer room settings in the wizard can be preselected via UCR variables
→ TODO

> * A confirmation dialog is missing for finishing/collecting an exam
→ OK

> * "Stop exam" → "Finish exam"
→ OK

> * The order of "Finish exam" and "Collect exam" can be swapped (maybe "Finish
> exam" with red text color?)
→ OK

> * The progress descriptions for finishing an exam are slightly incorrect
→ TODO

> * Room settings should rather be set to their default values after finishing an
> exam
→ OK

> * Check whether "examDescription" is correctly handed over to SettingsDialog
→ have been removed (→ Comment #2)

(In reply to comment #2)
> (In reply to comment #1)
> > ...
> > * The order of "Finish exam" and "Collect exam" can be swapped (maybe "Finish
> > exam" with red text color?)
> 
> → maybe an icon would even be better than color?
→ OK

> > ...
> > * Check whether "examDescription" is correctly handed over to SettingsDialog
> 
> → I think this can be removed from the SettingsDialog, as the description can
> be fetched from the project file in the backend
→ OK

QA TODO: everything after comment2 and the two TODO's from above
Comment 16 Alexander Kläser univentionstaff 2013-05-22 12:52:57 CEST
(In reply to comment #15)
> (In reply to comment #0)
> > The following points have not yet been implemented with Bug 30831:
> > * Warn if an exam project already exists
> → REOPEN
> the error message is OK but the room is then displayed as 'writes an exam' (a
> new exam really starts)

→ TODO

> ...
> > * Read the max upload size from UCRV umc/server/upload/max and configure the
> > MultiUploader accordingly
> → OK
> REOPEN: if i 1. add files 2. create a exam 3. end exam 4. create new exam: the
> files from exam1 are in the file list (not removed) 

As noted in the text, see Bug 31318.

> > * JS-side validation of user input
> → OK
> REOPEN: missing red exclamation mark when an invalid value is set

I forgot to refer to Bug 31337.

> (In reply to comment #1)
> > More points:
> > * "Directory name" goes on the second page of the wizard
> → OK
> → Is it OK that the name also changes if it was already edited by hand?

I'd say yes, it is OK for now. We should change it if we get feedback.

> > * Computer room settings in the wizard can be preselected via UCR variables
> → TODO

This has been done, I forgot to mention it… the variables are defined in the package:

[ucsschool/exam/default/room]
Description[en]=Specifies the preselected room for a new exam. The value is given as the LDAP name of the room, i.e., including the school OU prefix (e.g., "myschool-PC Room").
...
[ucsschool/exam/default/shares]
Description[en]=Specifies the preselected share mode for a new exam. Possible values are "all" for access to all shares without restriction and "home" for restricted access only to the (exam user) home directory.
...
[ucsschool/exam/default/internet]
Description[en]=Specifies the preselected internet rule for a new exam. Possible values are names of internet rules as they are specified in the UMC module "Define internet rules".
...

> > * The progress descriptions for finishing an exam are slightly incorrect
> → TODO

Has been fixed. Have you seen problems?
Comment 17 Alexander Kläser univentionstaff 2013-05-23 18:40:12 CEST
(In reply to comment #15)
> (In reply to comment #0)
> > The following points have not yet been implemented with Bug 30831:
> > * Warn if an exam project already exists
> → REOPEN
> the error message is OK but the room is then displayed as 'writes an exam' (a
> new exam really starts)

→ I could not reproduce this error.


The following points have been addressed:
* When finishing an exam, the home directories of exam user accounts are now removed, as well.
* In the list of all available computers, computers on which a teacher is logged in are not rebooted.


 ucs-school-umc-exam (1.0.8-1) unstable; urgency=low
 .
   * Bug #31158: remove exam user home, filter out teacher computers for reboot
Comment 18 Alexander Kläser univentionstaff 2013-05-27 11:22:46 CEST
Some usability changes for rebooting the computers need yet to be adapted.
Comment 19 Alexander Kläser univentionstaff 2013-05-27 12:45:05 CEST
(In reply to comment #14)
> ...
> (In reply to comment #12)
> > After creating a new exam session, the UMC module reports:
> > 
> >    Die Vorbereitung der Klassenarbeit war erfolgreich.
> > 
> > but within the logfile the following errors are found (due to bug 31389):
> > 
> > 17.05.13 05:09:47.233  MODULE      ( WARN    ) : Could not create exam user
> > account for: uid=tanja516,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local
> > 17.05.13 05:09:47.233  MODULE      ( PROCESS ) : Vorbereitung der
> > Klassenarbeitskonten - Koch, Ulrich (ulrich514)
> > 17.05.13 05:09:47.397  MODULE      ( WARN    ) : Could not create exam user
> > account for: uid=ulrich514,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local
> > 
> > The UMC module should show occurent errors.
> 
> This should be discussed in Bug 31389.

As it has not been discussed there, Bug 31516 has been created.
Comment 20 Florian Best univentionstaff 2013-05-27 15:55:58 CEST
(In reply to comment #7)
> More adjustments for the computer room:
> * More specific error handling with information when a room is in use and/or in
> exam mode
→ OK

> * Error handling when stopping an exam in case no files have been distributed
> or the project file does not exist anymore
→ I removed the project files in /var/lib/ucs-school-umc-schoolexam but I did not get any error message, is this correct?

(In reply to comment #10)
> (In reply to comment #9)
> > (In reply to comment #8)
> > > ...
> > > valueRequired: Die Eigenschaft Primäre Gruppe wird benötigt.
> > 
> > This happens if for any reason a user is member of a group but the user object
> > has been removed. I observed exam users (which have been removed) being present
> > in a class group. A try except block could be used to mask this problem.
> > However, I am not sure yet what is the source of the problem.
> 
> This problem should be fixed:
> 
> ucs-school-umc-distribution (8.0.9-1) unstable; urgency=low
> .
>  * Bug #31158: make sure that not existing user objects are ignored
→ OK

> The following points have been addressed, as well:
> * When stopping an exam, the computer room is reset to its default values
→ OK

> * An exam user account is only removed if it is not in use in a parallel exam
> (could not be tested extensively due to Bug 31389)
→ OK

(In reply to comment #14)
> (In reply to comment #11)
> > I read all dialogs of the exam module carefully (it took some time) and in the
> > end when I tried to start the exam, I got the following error message:
> > 
> > "Ein unerwarteter Fehler trat währende der Vorbereitung auf: {'desc': "Can't
> > contact LDAP server"}"
> > 
> > Even I press "Weiter" and try it again, it does not work. Same error message.
> 
> This is due to an LDAP timeout that happened in the thread function. I moved
> the first LDAP operation outside the thread such that the LDAP_Connection
> wrapper can retry a second time.
> 
> fixed in → ucs-school-umc-exam (0.1.33-1)
→ OK
 
> (In reply to comment #12)
> > After creating a new exam session, the UMC module reports:
> > 
> >    Die Vorbereitung der Klassenarbeit war erfolgreich.
> > 
> > but within the logfile the following errors are found (due to bug 31389):
> > 
> > 17.05.13 05:09:47.233  MODULE      ( WARN    ) : Could not create exam user
> > account for: uid=tanja516,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local
> > 17.05.13 05:09:47.233  MODULE      ( PROCESS ) : Vorbereitung der
> > Klassenarbeitskonten - Koch, Ulrich (ulrich514)
> > 17.05.13 05:09:47.397  MODULE      ( WARN    ) : Could not create exam user
> > account for: uid=ulrich514,cn=schueler,cn=users,ou=gsmitte,dc=nstx,dc=local
> > 
> > The UMC module should show occurent errors.
> 
> This should be discussed in Bug 31389.
→ OK → comment #19
 
> (In reply to comment #13)
> > * On the exam reboot page, there is no initial explanation why all computers
> >   have to be rebooted.
> 
> Good point, fixed in ucs-school-umc-exam (1.0.1-0).
→ OK
→ REOPEN: german translation typo "Nurdurch" → "Nur durch"

> > * Which item/widget indicates that a connection to a windows client has bee 
> >   established successfully?
> 
> The icon, but it is certainly not explicit enough, I added a new column in
> ucs-school-umc-exam (1.0.1-0).
> 
> Also added the summary of the exam configuration in ucs-school-umc-exam
> (1.0.1-0).
→ OK

(In reply to comment #17)
> (In reply to comment #15)
> > (In reply to comment #0)
> > > The following points have not yet been implemented with Bug 30831:
> > > * Warn if an exam project already exists
> > → REOPEN
> > the error message is OK but the room is then displayed as 'writes an exam' (a
> > new exam really starts)
> 
> → I could not reproduce this error.
I could reproduce this again → REOPEN

> The following points have been addressed:
> * When finishing an exam, the home directories of exam user accounts are now
> removed, as well.
→ OK
(This could lead to errors on the students computers)

> * In the list of all available computers, computers on which a teacher is
> logged in are not rebooted.
→ REOPEN: why don't we reboot only computers where a student is logged in?
Administrators and Staff are also rebooted due to this. From my school experience this really happens!
The select boxes are irritating for me because I don't know what happens if I select only some computers and click on reboot.
Comment 21 Florian Best univentionstaff 2013-05-27 16:07:14 CEST
(In reply to comment #16)
→ OK

> > > * Computer room settings in the wizard can be preselected via UCR variables
> > → TODO
> 
> This has been done, I forgot to mention it… the variables are defined in the
> package:
> ...
→ REOPENED: The variables are not registered. Missing univention-install-config-registry-info in override_dh_auto_install.

> > > * The progress descriptions for finishing an exam are slightly incorrect
> > → TODO
> 
> Has been fixed. Have you seen problems?
→ OK

The TODO's were only for me to remember what wasn't already QAed.
Comment 22 Alexander Kläser univentionstaff 2013-05-27 17:22:20 CEST
Some more points have been addressed:
* Reboot page: now, it exists only one 'next' button, on pressing a dialogue for rebooting is prompted
* Help texts have been adapted to make them shorter and more precise.
* Added hint that students need to login with a special exam account.
* The grid for rebooting has been rendered larger.
* The status information for (reboot) grid entries have been adjusted to be more precise.

 ucs-school-umc-exam (1.0.11-1) unstable; urgency=low
 .
   * Bug #31158: adjusted help texts, usability changes for computer rebooting,
     larger RebootGrid
Comment 23 Alexander Kläser univentionstaff 2013-05-27 19:16:15 CEST
(In reply to comment #20)
> ...
> > * Error handling when stopping an exam in case no files have been distributed
> > or the project file does not exist anymore
> → I removed the project files in /var/lib/ucs-school-umc-schoolexam but I did
> not get any error message, is this correct?

Yes, this behaviour was intended.

Previously, I received a traceback when 0 files were in the project or when the project file did not exist. Currently, it is possible to mark an error during the progress of collecting an exam (due to limitations in the progress bar). It is not directly possible to mark the process as success with some additional warnings. As it is crucial to be able to finish an exam, I chose this implementation logic.

I opened Bug 31521 for further considerations.

> ...
> > (In reply to comment #13)
> > > * On the exam reboot page, there is no initial explanation why all computers
> > >   have to be rebooted.
> > 
> > Good point, fixed in ucs-school-umc-exam (1.0.1-0).
> → OK
> → REOPEN: german translation typo "Nurdurch" → "Nur durch"

OK, this part does not exist any longer.

> ...
> (In reply to comment #17)
> > (In reply to comment #15)
> > > (In reply to comment #0)
> > > > The following points have not yet been implemented with Bug 30831:
> > > > * Warn if an exam project already exists
> > > → REOPEN
> > > the error message is OK but the room is then displayed as 'writes an exam' (a
> > > new exam really starts)
> > 
> > → I could not reproduce this error.
> I could reproduce this again → REOPEN

True, I could reproduce it when starting a second exam with the same name. I would get an error in the exam module, however, the corresponding computer room is marked as writing the exam XYZ, i.e., it is written in two rooms simultaneously. This can lead to inconsistencies.

Note if the exam project files is deleted, the same exam can be written twice. However, this behaviour is expected. Each computer room is saving its state of writing an exam in a room file (/var/cache/ucs-school-umc-computerroom), i.e., separately from the exam project file (/var/lib/ucs-school-umc-exam).

The problem was that the chosen computer room was set into exam mode before evaluating possible errors. This has been adapted now, the exam mode is set after creating exam user accounts.

> ...
> > * In the list of all available computers, computers on which a teacher is
> > logged in are not rebooted.
> → REOPEN: why don't we reboot only computers where a student is logged in?
> Administrators and Staff are also rebooted due to this. From my school
> experience this really happens!

The current information allows to distinguish between teacher and not teacher. Yes, this could be changed. I would suggest to create an enhancement bug for the computer room module in this case :) .

> The select boxes are irritating for me because I don't know what happens if I
> select only some computers and click on reboot.

This is also correct. I would opt for dynamically detecting whether actions are set or not. This is a generic issue that should be addressed separately. I created Bug 31522 for this.

(In reply to comment #21)
> (In reply to comment #16)
> > > > * Computer room settings in the wizard can be preselected via UCR variables
> > > → TODO
> > 
> > This has been done, I forgot to mention it… the variables are defined in the
> > package:
> > ...
> → REOPENED: The variables are not registered. Missing
> univention-install-config-registry-info in override_dh_auto_install.

Correct, thanks, I adjusted the debian/rules file for this.


 ucs-school-umc-exam (1.0.12-1) unstable; urgency=low
 .
   * Bug #31158: fixed order of ucmp commands when initiating a new exam, fixed
     translations, fixed registration of UCR variables
Comment 24 Florian Best univentionstaff 2013-05-28 09:18:30 CEST
(In reply to comment #22)
> Some more points have been addressed:
> * Reboot page: now, it exists only one 'next' button, on pressing a dialogue
> for rebooting is prompted
→ OK, maybe the 'Cancel'-button should be labeled as "Don't restart"
> * Help texts have been adapted to make them shorter and more precise.
→ OK
> * Added hint that students need to login with a special exam account.
→ OK
> * The grid for rebooting has been rendered larger.
→ OK
> * The status information for (reboot) grid entries have been adjusted to be
> more precise.
→ OK

(In reply to comment #23)
> (In reply to comment #20)
> > ...
> > > * Error handling when stopping an exam in case no files have been distributed
> > > or the project file does not exist anymore
> > → I removed the project files in /var/lib/ucs-school-umc-schoolexam but I did
> > not get any error message, is this correct?
> 
> Yes, this behaviour was intended.
> …
> I opened Bug 31521 for further considerations.
→ OK

> > → REOPEN: german translation typo "Nurdurch" → "Nur durch"
> 
> OK, this part does not exist any longer.
→ OK

> 
> > ...
> > (In reply to comment #17)
> > > (In reply to comment #15)
> > > > (In reply to comment #0)
> > > > > The following points have not yet been implemented with Bug 30831:
> > > > > * Warn if an exam project already exists
> > > > → REOPEN
> > > > the error message is OK but the room is then displayed as 'writes an exam' (a
> > > > new exam really starts)
> > > 
> > > → I could not reproduce this error.
> > I could reproduce this again → REOPEN
> 
> True, I could reproduce it when starting a second exam with the same name. I
> would get an error in the exam module, however, the corresponding computer room
> is marked as writing the exam XYZ, i.e., it is written in two rooms
> simultaneously. This can lead to inconsistencies.
> 
> Note if the exam project files is deleted, the same exam can be written twice.
> However, this behaviour is expected. Each computer room is saving its state of
> writing an exam in a room file (/var/cache/ucs-school-umc-computerroom), i.e.,
> separately from the exam project file (/var/lib/ucs-school-umc-exam).
> 
> The problem was that the chosen computer room was set into exam mode before
> evaluating possible errors. This has been adapted now, the exam mode is set
> after creating exam user accounts.
→ OK


> > > * In the list of all available computers, computers on which a teacher is
> > > logged in are not rebooted.
> > → REOPEN: why don't we reboot only computers where a student is logged in?
> > Administrators and Staff are also rebooted due to this. From my school
> > experience this really happens!
> 
> The current information allows to distinguish between teacher and not teacher.
> Yes, this could be changed. I would suggest to create an enhancement bug for
> the computer room module in this case :) .

> > The select boxes are irritating for me because I don't know what happens if I
> > select only some computers and click on reboot.
> 
> This is also correct. I would opt for dynamically detecting whether actions are
> set or not. This is a generic issue that should be addressed separately. I
> created Bug 31522 for this.
→ OK

> > → REOPENED: The variables are not registered. Missing
> > univention-install-config-registry-info in override_dh_auto_install.
> 
> Correct, thanks, I adjusted the debian/rules file for this.
→ OK
Comment 25 Florian Best univentionstaff 2013-05-28 09:26:09 CEST
REOPENED: typos

s/acces/access/
s/immediatley/immediately/
s/The folgenden Sonderzeichen/Die folgenden Sonderzeichen/

empty msgstr:
"Replication timeout: could not create all exam users"
Comment 26 Alexander Kläser univentionstaff 2013-05-28 11:13:51 CEST
(In reply to comment #25)
> REOPENED: typos
> 
> s/acces/access/
> s/immediatley/immediately/
> s/The folgenden Sonderzeichen/Die folgenden Sonderzeichen/
> 
> empty msgstr:
> "Replication timeout: could not create all exam users"

Thanks! I fixed the typos.

 ucs-school-umc-exam (1.0.13-1) unstable; urgency=low
 .
   * Bug #31158: fixed typos
Comment 27 Florian Best univentionstaff 2013-05-28 11:39:55 CEST
(In reply to comment #26)
> (In reply to comment #25)
> > REOPENED: typos
> > 
> > s/acces/access/
> > s/immediatley/immediately/
> > s/The folgenden Sonderzeichen/Die folgenden Sonderzeichen/
> > 
> > empty msgstr:
> > "Replication timeout: could not create all exam users"
> 
> Thanks! I fixed the typos.
> 
>  ucs-school-umc-exam (1.0.13-1) unstable; urgency=low
>  .
>    * Bug #31158: fixed typos
→ OK

I added the bug number to the generic exam-mode changelog entry.

→ VERIFIED
Comment 28 Florian Best univentionstaff 2013-05-28 16:33:06 CEST
When closing the browser window while the user profiles get copied we get into an inconsistent state:
The exam starts (profile file gets created) but the computerroom module does not know about this because it will not get acquired.
→ The computerroom module should be acquired in the backend when starting the exam
Comment 29 Alexander Kläser univentionstaff 2013-05-28 17:38:31 CEST
(In reply to comment #28)
> When closing the browser window while the user profiles get copied we get into
> an inconsistent state:
> The exam starts (profile file gets created) but the computerroom module does
> not know about this because it will not get acquired.
> → The computerroom module should be acquired in the backend when starting the
> exam

Thanks for the hint, this is correct. I moved this operation into the python module to render it more robust.


 ucs-school-umc-exam (1.0.17-1) unstable; urgency=low
 .
   * Bug #31158: moved adjustment of room settings into python module
Comment 30 Florian Best univentionstaff 2013-05-29 08:54:46 CEST
(In reply to comment #29)
> Thanks for the hint, this is correct. I moved this operation into the python
> module to render it more robust.
→ OK
Comment 31 Florian Best univentionstaff 2013-05-29 13:19:18 CEST
When starting a new exam the teacher-computer was displayed as 'reboot required'. When clicking on 'reboot' it did not restart so only the display is wrong.
Comment 32 Alexander Kläser univentionstaff 2013-05-29 16:11:01 CEST
(In reply to comment #31)
> When starting a new exam the teacher-computer was displayed as 'reboot
> required'. When clicking on 'reboot' it did not restart so only the display is
> wrong.

fixed.

 ucs-school-umc-exam (1.0.19-1) unstable; urgency=low
 .
   * Bug #31158: adjust label of teacher computers in grid, enhance labeling of
     buttons/columns, make sure monitoring is re-evaluated when going on the
     reboot page again
Comment 33 Florian Best univentionstaff 2013-05-29 16:45:47 CEST
(In reply to comment #32)
→ OK
Comment 34 Florian Best univentionstaff 2013-05-30 11:56:59 CEST
To reset the computer settings it is required to reboot (or shutdown) the computers again. Therefore in the finish exam dialogue a notification should be implemented.
Comment 35 Alexander Kläser univentionstaff 2013-05-30 13:43:40 CEST
(In reply to comment #34)
> To reset the computer settings it is required to reboot (or shutdown) the
> computers again. Therefore in the finish exam dialogue a notification should be
> implemented.

fixed.

 ucs-school-umc-computerroom (3.0.12-1) unstable; urgency=low
 .
   * Bug #31158: added note to restart/switch off computers after an exam
Comment 36 Florian Best univentionstaff 2013-05-30 14:24:00 CEST
(In reply to comment #35)
>  ucs-school-umc-computerroom (3.0.12-1) unstable; urgency=low
>  .
>    * Bug #31158: added note to restart/switch off computers after an exam
OK
Comment 37 Sönke Schwardt-Krummrich univentionstaff 2013-06-07 21:39:59 CEST
UCS@school 3.1 R2 has been released:
http://download.univention.de/doc/release-notes-ucsschool-3.1-rev2.pdf

If this error occurs again, please use "Clone This Bug".