Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Some attributes cannot be deleted by editing contact in roundcube #318

Closed
mstilkerich opened this issue Nov 15, 2020 · 2 comments
Closed
Assignees
Labels
Milestone

Comments

@mstilkerich
Copy link
Owner

For unset attributes, roundcube either provides us we an empty value or no entry at all in the given $save_data array. This seems to somewhat depend on whether an attribute is shown in the edit mask or not.

Currently, when saving a value a property will only be removed from the VCard if an empty string is given by roundcube for single-value properties. If no value is given at all, the previous value will be retained. This results in the problem that some attributes can not be deleted from roundcube, for example spouse, by using the delete button. As a workaround, it is possible to not delete the attribute in the edit mask but store an empty value instead, but this is highly non-intuitive for the user.

@mstilkerich mstilkerich self-assigned this Nov 15, 2020
@mstilkerich mstilkerich added this to the 4.1.0 milestone Nov 15, 2020
@mstilkerich
Copy link
Owner Author

mstilkerich commented Nov 15, 2020

Note that PHOTO is special: Roundcube will

  • give no value (photo key not set in $save_data) in case the photo was left unchanged
  • give a value as binary string if the photo was set/changed
  • give an empty string as value if the photo was deleted

mstilkerich added a commit that referenced this issue Nov 15, 2020
Except for PHOTO, which in this case must retain its original value.
@mstilkerich
Copy link
Owner Author

Fixed on 4.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant