[ADD] allow adding calendar links to time off calendar, default icals

[hbrunn]

in OCA/server-backend#231 habe ich ein auto-flag hinzugefügt, mit dem ein Kalender fur alle user aktiviert wird (sie können das von Hand wieder deaktivieren).

Hier gibt es ein flag um bestimmte Kalender im Leave-Kalender anzuzeigen, die Links erscheinen dann unter der Legende.

Weiterhin definiere ich zwei Standardkalender die automatisch aktiviert sind und in der Kalender-Ansicht angezeigt werden.

Diskussion:

1. 'Mein Team' habe ich jetzt definiert als 'In meiner Abteilung', das muss ggf noch verändert werden
2. Statt nur dem Link vielleicht besser wenn ein Click die URL in die Zwischenablage kopiert und das in einem Popup sagt?

[albig]

Ich wollte das mal wieder ausprobieren, scheitere an folgender Fehlermeldung, die mich leider ratlos macht:

odoo15-web15-1    |   File "/mnt/extra-addons/server-backend/base_ical/models/base_ical.py", line 191, in _get_ical
odoo15-web15-1    |     event.add(field.vevent_field).value = self._format_ical_value(
odoo15-web15-1    |   File "/mnt/extra-addons/server-backend/base_ical/models/base_ical.py", line 199, in _format_ical_value
odoo15-web15-1    |     return pytz.utc.localize(value).astimezone(pytz.timezone(self.env.user.tz))
odoo15-web15-1    |   File "/usr/lib/python3/dist-packages/pytz/__init__.py", line 170, in timezone
odoo15-web15-1    |     if zone.upper() == 'UTC':
odoo15-web15-1    | AttributeError: 'bool' object has no attribute 'upper'

[hbrunn]

das sieht aus wie ein Benutzer ohne gesetzte Zeitzone - das sollte bei Euch ohnehin default Europe/Berlin sein denke ich? (sowas könnte Teil der Defaults/Templates sein die wir ohnehin erstellen wollen)

[albig]

Der Benutzer hat die richtige Zeitzone. Das passiert, wenn man den Modul verdigado_attendance upgraden will.

[hbrunn]

gilt das auch fur den system-user (system, inaktiv)?

[hbrunn]

habe ein Fallback für nicht konfigurierte Zeitzonen eingebaut. Aber falls der system user keine Zeitzone hat sollte er die gesetzt bekommen, ansonsten läuft mancher Code in der falschen Zeitzone

[albig]

Das war's tatsächlich. Der User __system__ ist ja ausgeblendet und hatte keine Zeitzone.

Jetzt scheitert's hier:

odoo15-web15-1    |   File "/usr/lib/python3/dist-packages/vobject/base.py", line 1219, in backslashEscape
odoo15-web15-1    |     s = s.replace("\\", "\\\\").replace(";", "\;").replace(",", "\,")
odoo15-web15-1    | AttributeError: 'bool' object has no attribute 'replace'
odoo15-web15-1    |
odoo15-web15-1    | The above exception was the direct cause of the following exception:

odoo15-web15-1    |   File "/usr/lib/python3/dist-packages/odoo/tools/convert.py", line 696, in _tag_root
odoo15-web15-1    |     raise ParseError('while parsing %s:%s, somewhere inside\n%s' % (
odoo15-web15-1    | odoo.tools.convert.ParseError: while parsing /mnt/extra-addons/verdigado_attendance/data/base_ical.xml:16, somewhere inside
odoo15-web15-1    | <record id="ical_my_team_leaves" model="base.ical">
odoo15-web15-1    |         <field name="name">My team leaves</field>
odoo15-web15-1    |         <field name="model_id" ref="hr_holidays.model_hr_leave"/>

Das lief doch schon mal...?

[hbrunn]

das ist abhängig von den bestehenden Daten, ich habe jetzt in den Ausdrücken für das name-Feld ein or '' zugefügt, das sollte diesen Fehler beheben

[albig]

Danke, auch das läuft jetzt.

Jetzt möchte ich "My leaves" bearbeiten unter Technisches -> iCalendars. Das endet in einem JavaScript-Fehler.

web.assets_backend.min.js:2951 Uncaught (in promise) Error: NameError: name 'user' is not defined
    at PY_ensurepy (web.assets_backend.min.js:2951:65)
    at py.evaluate (web.assets_backend.min.js:3068:8)
    at py.evaluate (web.assets_backend.min.js:3080:25)
    at py.evaluate (web.assets_backend.min.js:3075:111)
    at py.evaluate (web.assets_backend.min.js:3077:99)
    at py.eval (web.assets_backend.min.js:3082:284)
    at web.assets_backend.min.js:3881:136
    at _.each._.forEach (web.assets_common.min.js:153:404)
    at _.<computed> [as each] (web.assets_common.min.js:218:498)
    at eval_domains (web.assets_backend.min.js:3880:265)

Unter "Domain" steht nur "Diese Domain wird nicht unterstützt.".

[hbrunn]

ah, deshalb habe ich in der ersten Version das domain-widget nicht verwendet. Das wieder zurück gebaut

[albig]

Noch zwei Fragen bzw. Änderungswünsche.

[albig]

Bei "My Team Leaves" bitte nur den Namen, keinen Grund oder ähnliches.

Es scheint zu funktionieren:
<field name="expression_summary">record.employee_id.name or ''</field>

Kann man den Abwesenheits-Typ hier ermitteln? Also dass dann z.B. steht "Alexander Bigga: Urlaub", wenn der Abwesenheits-Typ "Urlaub" (original: "Bezahlte Abwesenheit") lautet?

[hbrunn]

'%s: %s' % (record.employee_id.name, record.holiday_status_id.name)

[albig]

Super, wenn Du das noch hier änderst, dann merg ich das.

[hbrunn]

domain im Teamkalender angepasst und die namen verändert

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b25ba7a) 71.14% compared to head (0235eb8) 71.84%.
Report is 2 commits behind head on 15.0.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             15.0      #26      +/-   ##
==========================================
+ Coverage   71.14%   71.84%   +0.70%     
==========================================
  Files          10       11       +1     
  Lines         201      206       +5     
  Branches        5        5              
==========================================
+ Hits          143      148       +5     
  Misses         57       57              
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.