Issue Details (XML | Word | Printable)

Key: CONTINUUM-1130
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Emmanuel Venisse
Reporter: Brett Porter
Votes: 5
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
Continuum

can't delete default project group

Created: 09/Jan/07 09:53 AM   Updated: 30/Mar/09 11:57 PM   Resolved: 30/Mar/09 11:57 PM
Return to search
Component/s: Core system, Database
Affects Version/s: 1.1
Fix Version/s: 1.3.3

Time Tracking:
Not Specified

File Attachments: 1. Text File CONTINUUM-1130.patch (6 kB)


Complexity: Intermediate


 Description  « Hide

I don't really want/need this - but after deleting it, it keeps coming back



Jesse McConnell made changes - 12/Mar/07 01:28 PM
Field Original Value New Value
Fix Version/s Future [ 13292 ]
George Gastaldi added a comment - 04/Oct/07 09:50 PM

It is still a problem in version 1.1-beta-3.


Wendy Smoak added a comment - 23/Nov/07 09:14 AM

In 1.1 Final, attempting to delete the Default project group results in:

Error Occurred
javax.jdo.JDOUserException: One or more instances could not be deleted NestedThrowables: javax.jdo.JDODataStoreException: Delete request failed: DELETE FROM BUILDDEFINITION WHERE ID = ? NestedThrowables: SQL Exception: DELETE on table 'BUILDDEFINITION' caused a violation of foreign key constraint 'BUILDDEFINITCE_FK2' for key (3). The statement has been rolled back.

Show/hide Stack Trace

        javax.jdo.JDOUserException: One or more instances could not be deleted
	at org.jpox.AbstractPersistenceManager.deletePersistentAll(AbstractPersistenceManager.java:1525)
	at org.jpox.store.rdbms.scostore.ElementContainerStore.clear(ElementContainerStore.java:662)
...
NestedThrowablesStackTrace:
javax.jdo.JDODataStoreException: Delete request failed: DELETE FROM BUILDDEFINITION WHERE ID = ?
	at org.jpox.store.rdbms.request.DeleteRequest.execute(DeleteRequest.java:288)
	at org.jpox.store.rdbms.table.ClassTable.delete(ClassTable.java:2611)
	at org.jpox.store.StoreManager.delete(StoreManager.java:981)
...
NestedThrowablesStackTrace:
ERROR 23503: DELETE on table 'BUILDDEFINITION' caused a violation of foreign key constraint 'BUILDDEFINITCE_FK2' for key (3).  The statement has been rolled back.
	at org.apache.derby.iapi.error.StandardException.newException(Unknown Source)
	at org.apache.derby.impl.sql.execute.ReferencedKeyRIChecker.doCheck(Unknown Source)
	at org.apache.derby.impl.sql.execute.RISetChecker.doPKCheck(Unknown Source)
...

Wendy Smoak made changes - 23/Nov/07 09:15 AM
Affects Version/s 1.1-alpha-1 [ 12082 ]
Affects Version/s 1.1 [ 13605 ]
Fix Version/s Future [ 13292 ]
Brett Porter made changes - 01/Apr/08 11:33 PM
Fix Version/s 1.2 [ 13779 ]
Rahul Thakur made changes - 02/Apr/08 05:20 PM
Assignee Rahul Thakur [ rahul ]
Rahul Thakur added a comment - 02/Apr/08 05:22 PM

Ok, I could be wrong but IIRC the idea of having a default Project Group was that no Projects would live outside a ProjectGroup.

Should we allow renaming the default ProjectGroup instead of deleting it?


Brett Porter added a comment - 02/Apr/08 05:25 PM

I would say only allow deleting it if it is empty, and there are no other groups. That way adding a project can always require addiion to an existing project group


Brett Porter made changes - 07/May/08 09:02 AM
Priority Major [ 3 ] Minor [ 4 ]
Fix Version/s 1.2 [ 13779 ]
Fix Version/s 1.3 [ 14168 ]
Maria Catherine Tan made changes - 28/Sep/08 07:00 PM
Link This issue is depended upon by CONTINUUM-1857 [ CONTINUUM-1857 ]
Maria Catherine Tan made changes - 28/Sep/08 07:01 PM
Link This issue is depended upon by CONTINUUM-1857 [ CONTINUUM-1857 ]
Jose Morales Martinez added a comment - 03/Feb/09 11:49 AM

The problem is not because it is the default project group, It's due to the fact project group contains a template definition. I obtain the same error by creating new project group and assign a template definition.

I got that Continuum can eliminate this type of projectgroup, but when they have problems with the generation of projects and Ant Console. Only need create a not null restriction about group project.

You believe that it would be working on this improvement, or just perform a single validation, so as not to allow the removal of the default project group?


Jose Morales Martinez added a comment - 10/Feb/09 07:42 PM

This patch allow delete default project group and project groups which have associated build templates. It also validates that when we add an Ant or Shell project, it must be in a project group .

Note: When we restart Continuum application, it creates the default project, I am working for to do that only the first time it create default project. Anyway, someone has a suggestion?


Jose Morales Martinez made changes - 10/Feb/09 07:42 PM
Attachment CONTINUUM-1130.patch [ 39950 ]
Emmanuel Venisse made changes - 23/Mar/09 06:01 AM
Assignee Rahul Thakur [ rahul ] Emmanuel Venisse [ evenisse ]
Fix Version/s 1.3.x [ 14168 ]
Fix Version/s 1.3.3 [ 15105 ]
Emmanuel Venisse added a comment - 26/Mar/09 01:00 AM

This patch delete the Default Project Group but we have an exception in logs.


Emmanuel Venisse added a comment - 30/Mar/09 11:57 PM

Fixed in rev.760328
Now it is possible to remove the default project group without exception in logs. When a group is created the build definition template is cloned so th exception won't appear in the future.


Emmanuel Venisse made changes - 30/Mar/09 11:57 PM
Resolution Fixed [ 1 ]
Status Open [ 1 ] Closed [ 6 ]