davidcoutadeur

Results 83 comments of davidcoutadeur

This feature should be ready. Only thing missing is updating the requirement of smarty4. It can be done in https://github.com/ltb-project/self-service-password/issues/850

This feature is now provided in #865

I have tested extensively the feature. The only bug I encounter sometimes is that the field that is modified is not the custompwdfield, but the userPassword. I suspect there is...

> maybe it is because of `action="#"` in changecustompwd.tpl... I dont know if `&custompwdindex=N` is also submitted there, but should be. Maybe changing `action="#"` to `action=""` (although this SHOULD not...

> We also need to see if Smarty is a system dependency or a composer dependency smarty4 is a system dependency on debian. @xavierba is working on a smarty4 package...

NOTE: When (if) updating requirement to smarty4, we must also list the new version in deb package (replacement of smarty path in config files in `debian/replace_cache_dirs` + Depends: in `control`...

smarty4 is not available on debian 11 and debian 10. We may require it with as a OR dependency: `smarty4|smarty3` (with a preference for smarty4)

For reference, there is a note here in the documentation: https://self-service-password.readthedocs.io/en/stable/installation.html#debian-ubuntu The debian smarty3 package seems compatible

Thanks for the pointer. When the fix is available on all supported plaforms of self-service-password, we may add the dependency again...

Seems nearly finished now. Need to do the equivalent task for debian: #852 , then we can merge