Bug #3661

QubitObject -> transient objects issue

Added by Evelyn McLellan over 11 years ago. Updated almost 6 years ago.

Status:NewStart date:
Priority:MediumDue date:
Assignee:-% Done:

0%

Category:-
Target version:-
Google Code Legacy ID:atom-1712 Tested version:
Sponsored: Requires documentation:

Description

To reproduce this error: ========================
1)Create or edit repository record
2)Select one or more languages and scripts in control area
3)Save the record

Resulting error: ================
Languages and scripts don't save

[g] Legacy categories: Form validation


Related issues

Duplicated by Access to Memory (AtoM) - Bug #3597: Can't edit languages (ISAAR) Duplicate
Duplicated by Access to Memory (AtoM) - Bug #3651: When creating multiple new subject or place terms in info... Duplicate

History

#1 Updated by Evelyn McLellan over 11 years ago

  • Subject set to Language(s) and Script(s) not saving in ISDIAH, ISAAR or ISDF control area

They don't save in ISAAR or ISDF records either. Changing issue title to reflect this.

#3 Updated by Jesús García Crespo over 11 years ago

See r7002.

[g] New owner: David Juhasz

#4 Updated by Jesús García Crespo over 11 years ago

ISAD is also affected. It is not reproducible when creating, only editing.

#6 Updated by Jesús García Crespo over 11 years ago

  • Priority changed from Medium to High

QubitObject properties are not being updated because $this->propertys-transient always return value zero. This is making impossible to update many fields set as properties in RAD and probably other templates. Please, see r7002.

[g] Labels added: Priority-High
[g] Labels removed: Priority-Medium

#7 Updated by Jesús García Crespo over 11 years ago

  • Subject set to QubitObject properties are not updated (propertys->transient issue)

Summary updated.

#8 Updated by Jesús García Crespo over 11 years ago

  • Subject set to QubitObject -> "propertys" are not updated (transient issue)

#10 Updated by Jesús García Crespo over 11 years ago

  • Subject set to QubitObject -> transient objects issue

This code is also problematic: $this->objectTermRelationsRelatedByobjectId->transient (see /p/qubit-toolkit/issues/detail?id=1702).

#11 Updated by Evelyn McLellan over 11 years ago

  • Priority changed from High to Critical

This affects a lot of fields, particularly in RAD, so I'm upgrading the issue to critical.

[g] Labels added: Priority-Critical
[g] Labels removed: Priority-High

#12 Updated by Jesús García Crespo over 11 years ago

Temporary solution in r7767, drop transient because is causing many bugs.

We have to try to make transient work but probably not in release 1.1

From David Juhasz:
----------------------------------------------
I did a bunch of research into the reason why the language and script properties are not being updated properly. The problem in this case is that we are trying to update an existing property, and the $propertys->transient method only returns related properties that have been added to the record with the $this->resource->propertys[] = foo; syntax.

I don't have a good solution to the problem right now, but I would prefer to find some way to flag an updated property so it is saved when the record is saved. The reason I switched to using the $propertys->transient array is that updating all of the related propertys, events, etc. on every save of a resource is causing a significant slowdown, especially when importing a batch of records.

That said, we may have to revert to just looping through $resource->propertys for Release 1.1 and figure out a long-term fix for Post 1.1. Maybe if we switch to Doctrine for an ORM it will solve this problem for us. :-P

[g] Labels added: Performance

#13 Updated by David Juhasz over 11 years ago

  • Target version changed from Release 1.1 to Release 1.2

[g] Labels added: Milestone-Release-1.2
[g] Labels removed: Milestone-Release-1.1

#14 Updated by David Juhasz over 11 years ago

  • Priority changed from Critical to High

[g] Labels added: Priority-High
[g] Labels removed: Priority-Critical

#15 Updated by David Juhasz almost 11 years ago

[g] Labels added: Milestone-Release-Post-1.2
[g] Labels removed: Milestone-Release-1.2

#16 Updated by David Juhasz almost 11 years ago

  • Priority set to Medium

[g] Labels added: Priority-Medium

#17 Updated by Anonymous over 10 years ago

  • Target version set to Release 1.3

[g] Labels added: Milestone-Release-1.3

#18 Updated by David Juhasz almost 10 years ago

Reassign to new account.

[g] New owner: David Juhasz

#19 Updated by David Juhasz almost 6 years ago

  • Assignee deleted (David Juhasz)

Also available in: Atom PDF