crs4 / notredam

Automatically exported from code.google.com/p/notredam
http://www.notredam.org
17 stars 4 forks source link

Vocabulary - Class Editor - cannot re-edit a class #70

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Once a class has been created there is no way to modify its attributes. You 
have to delete it and re-create it from scratch.

Please, take the required actions so that as long as the class has no instances 
yet, it shall be possible to modify it.

Please, also consider that some modifications of the attributes should be 
possible even if the class has some instances: e.g. adding a new attribute, 
renaming an existing attribute.

Deleting an attribute of a class which has been instantiated is fine, as long 
as a warning is issued.
In general, it should not be possible to change attribute type (and other 
attributes' properties) of already instantiated classes.

Original issue reported on code.google.com by maurizio...@gmail.com on 12 Sep 2012 at 10:09

GoogleCodeExporter commented 9 years ago
We ran into this too.  This would be an awesome feature to have!

Original comment by t...@metaltoad.com on 4 Oct 2012 at 9:43

GoogleCodeExporter commented 9 years ago
* Cannot add attributes to classes when objects already exist

Steps to reproduce
Step 1: Add/edit a class
Step 2: Add an object
Step 3: Attempt to add a new attribute to the class

Result: Error - "Cannot add/remove attributes to/from a class which is already 
referenced in the knowledge base"
Expected result: The property would be inherited to all exiting objects

Original comment by t...@metaltoad.com on 4 Oct 2012 at 9:45

GoogleCodeExporter commented 9 years ago
The error it's correct. In this moment you cannot add/remove/edit attributes to 
classes when objects already exist.  But you can add/remove/edit attributes to 
classes in all other cases.

Original comment by fabrizio.solinas83@gmail.com on 5 Oct 2012 at 7:15

GoogleCodeExporter commented 9 years ago
At the moment there are still some issues in class attribute addition/removal 
(i.e., the underlying DB is updated correctly, but some parts of NotreDAM may 
not notice the changes until the whole application is restarted).  If you are 
interested, there are some notes in the source code and commit messages.

For now, the problem is mitigated with a check, which gives the error message 
you are seeing when you try to alter a class which is already instantiated, 
subclassed or referenced from other classes.

I should be able to fix this bug (and close this issue) in a few days.  Stay 
tuned.

Original comment by alce...@crs4.it on 5 Oct 2012 at 9:38

GoogleCodeExporter commented 9 years ago

Original comment by fabrizio.solinas83@gmail.com on 16 Oct 2012 at 1:45

GoogleCodeExporter commented 9 years ago
Thank you for the fix!  I really appreciate this, and I am excited to use it!!  
Thank you!!

Original comment by t...@metaltoad.com on 16 Oct 2012 at 1:51