Bug 54115 - Integrate expiration date into the school library
Integrate expiration date into the school library
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: Ucsschool-lib
UCS@school 4.4
Other Mac OS X 10.1
: P5 normal (vote)
: UCS@school 4.4 v9-errata
Assigned To: Tobias Wenzel
Ole Schwiegert
:
Depends on:
Blocks: 54116
  Show dependency treegraph
 
Reported: 2021-11-19 17:44 CET by Ole Schwiegert
Modified: 2021-11-29 07:49 CET (History)
1 user (show)

See Also:
What kind of report is it?: Feature Request
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 Ole Schwiegert univentionstaff 2021-11-19 17:44:28 CET

    
Comment 1 Ole Schwiegert univentionstaff 2021-11-19 18:13:35 CET
Implemented on oschwieg/54115

- Since it is only visible if chosen (optional field) and I did not include it in the default visible fields, no screenshots and documentation should need adaptation. (What is actually desired here?)
- This field has the same limitation as the birthday field in that it can not be unset, once it was set to a value (Does it need to be 'fixed' within the Dienstleistung?)
- I am not aware of any particular changes needed in the importer to accommodate the new field?
Comment 2 Tobias Wenzel univentionstaff 2021-11-23 16:41:47 CET
- Since it is only visible if chosen (optional field) and I did not include it in the default visible fields, no screenshots and documentation should need adaptation. (What is actually desired here?)

-> it is an optional field, so we decided no screenshots. we added a section, 
which mentions the ucr-v 
 ucsschool/wizards/schoolwizards/users/optional_visible_fields 
which is needed to set these.

- This field has the same limitation as the birthday field in that it can not be unset, once it was set to a value (Does it need to be 'fixed' within the Dienstleistung?)

-> we decided to fix this: So far we had this check in base.py:_alter_udm_obj

if value is not None and attr.map_to_udm

which leads to None values being ignored. 

The solution includes introducing map_if_none in the class Attribute. Birthday &  UserExpirationDate set the value to True, the default is False.


UCS: 4.4-8 errata1072
Installed: cups=2.2.1 samba4=4.10 squid=3.5 ucsschool=4.4 v9 


packages are merge & build in 4.4 and wait for jenkins


Package: ucs-school-lib
Version: 12.2.37A~4.4.0.202111231618
Branch: ucs_4.4-0
Scope: ucs-school-4.4


Package: ucs-school-import
Version: 17.0.74A~4.4.0.202111231620
Branch: ucs_4.4-0
Scope: ucs-school-4.4


Package: ucs-school-umc-wizards
Version: 11.0.4A~4.4.0.202111231623
Branch: ucs_4.4-0
Scope: ucs-school-4.4


Package: ucs-test-ucsschool
Version: 6.0.253A~4.4.0.202111231625
Branch: ucs_4.4-0
Scope: ucs-school-4.4
Comment 3 Tobias Wenzel univentionstaff 2021-11-24 09:48:54 CET
FYI: 

We did not add a new schoollib- test, because person.verify checks the expiration_date implicitly in the import tests.
Comment 4 Tobias Wenzel univentionstaff 2021-11-24 09:53:15 CET
fixed tests

Package: ucs-test-ucsschool
Version: 6.0.254A~4.4.0.202111240951
Branch: ucs_4.4-0
Scope: ucs-school-4.4
Comment 5 Ole Schwiegert univentionstaff 2021-11-25 08:06:36 CET
I had to add something I was missing to commit first. Translation for the UMC and proper optional handling.

Package: ucs-school-umc-wizards
Version: 11.0.6A~4.4.0.202111250805
Branch: ucs_4.4-0
Scope: ucs-school-4.4
Comment 6 Ole Schwiegert univentionstaff 2021-11-25 11:13:01 CET
Changelog&Advisory: OK
Manual&automatic tests: OK
Comment 7 Ole Schwiegert univentionstaff 2021-11-29 07:49:59 CET
Errata updates for UCS@school 4.4 v9 have been released.

https://docs.software-univention.de/changelog-ucsschool-4.4v9-de.html

If this error occurs again, please clone this bug.