After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 653233 - Ensure that FieldDetails are immutable to backends
Ensure that FieldDetails are immutable to backends
Status: RESOLVED OBSOLETE
Product: folks
Classification: Platform
Component: general
git master
Other All
: Normal minor
: Future
Assigned To: folks-maint
folks-maint
Depends on: 653679 653680 653682
Blocks:
 
 
Reported: 2011-06-23 14:03 UTC by Philip Withnall
Modified: 2018-08-04 08:28 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Philip Withnall 2011-06-23 14:03:36 UTC
As discussed with Alex on IRC, we should ensure (and document) that backends can't modify FieldDetails instances. Instead, they should replace instances of FieldDetails they want to change with new instances, which have been deep copied and modified as appropriate.

This means that clients can safely modify FieldDetails instances that they're handed, and re-use them when updating properties without having to worry about breaking the internal state of a backend.
Comment 1 Philip Withnall 2011-06-23 14:06:05 UTC
(In reply to comment #0)
> This means that clients can safely modify FieldDetails instances that they're
> handed, and re-use them when updating properties without having to worry about
> breaking the internal state of a backend.

Bzzt. This means that clients can safely *re-use FieldDetails instances as read-only* when updating properties.
Comment 2 Travis Reitter 2011-08-01 18:39:22 UTC
Punting bugs that won't be fixed by Folks 0.6.0.
Comment 3 GNOME Infrastructure Team 2018-08-04 08:28:36 UTC
-- GitLab Migration Automatic Message --

This bug has been migrated to GNOME's GitLab instance and has been closed from further activity.

You can subscribe and participate further through the new bug through this link to our GitLab instance: https://gitlab.gnome.org/GNOME/folks/issues/25.