Community
Participate
Working Groups
I would like to see (for advanced user) - Container - DirectResource - STATE - VIEW - READLOCK WRITELOCK - OID Simon
I added all I was described.. does someone has some needs ? The problem is I'm not sure If I used components in the way they are supposed to be used!! :-( I will provide a patch soon. Simon
That sounds good! It's not necessary but maybe we want to show the adapters list and some change subscription info?
(In reply to comment #2) > That sounds good! > > It's not necessary but maybe we want to show the adapters list and some change > subscription info? Good Idea!! I will also add the version of the revision, created, revised info if available (Proxy, TRANSIENT.. it is not available!) Simon
Oh yeah, that's interesting information, too! I also thought about a change history per object, but that's maybe something for later ;-)
Guys, I'm not sure to have understood this properly: I assume you are trying to show up all that information in the "Properties View", right? In that case, all that information would be nice from the development point of view, but I think some end-users might not be interested in seeing information in the property view when they are expecting values of a well-known package. In that case (IMHO), this enhancement could be enabled from the preferences panel, and set off by default. From the developer point of view, I think those are nice enhacements :)
(In reply to comment #5) > Guys, > > I'm not sure to have understood this properly: I assume you are trying to show > up all that information in the "Properties View", right? > > In that case, all that information would be nice from the development point of > view, but I think some end-users might not be interested in seeing information > in the property view when they are expecting values of a well-known package. > > In that case (IMHO), this enhancement could be enabled from the preferences > panel, and set off by default. > > From the developer point of view, I think those are nice enhacements :) > Hi Vic, Some of them no.. I agree. But it is why we put them into advanced users! :-) Also some will be useful for the users Session- Turn on/off passiveUpdate etc..etc.. WHat do you think ? Simon
> Hi Vic, > Some of them no.. I agree. But it is why we put them into advanced users! :-) Great! > Also some will be useful for the users > Session- Turn on/off passiveUpdate > etc..etc.. > WHat do you think ? Yes, that would be lovely! I've already seen some of your bugzillas and they look very interesting. Specially the locking feature :) The project I was working for just finished today (maintenance will start soon, though), so I think I could spend some time in UI enhancements :)
(In reply to comment #7) > > Hi Vic, > > Some of them no.. I agree. But it is why we put them into advanced users! :-) > > Great! > > > Also some will be useful for the users > > Session- Turn on/off passiveUpdate > > etc..etc.. > > WHat do you think ? > > Yes, that would be lovely! > I've already seen some of your bugzillas and they look very interesting. > Specially the locking feature :) > Locking is already checked-in and available in M3 !! :-) > The project I was working for just finished today (maintenance will start soon, > though), so I think I could spend some time in UI enhancements :) > Sounds good!! We should synchronized our effort to avoid duplicate work!
Re-assigning to Vik in preparation of his new committer state...
Already started... I will ask you for a review!
This bugzilla will only be for objects!
I will not have time to do this one. Vik, do you want to develop it. Simon
Sure! Have you already something done?
Yes but I was not proud of it! I went to a bad direction...
(In reply to comment #14) > Yes but I was not proud of it! I went to a bad direction... > So I deleted it long time ago...
No prob :P
Created attachment 121431 [details] patch v1 - incomplete - Added new option to UI Preferences Page, allows enabling/disabling debug property descriptors - Already showing OID and Container. Container doesn't seem to show text correctly (using adapterFactory). Icon is properly shown. Only remaning to add the missing IItemPropertyDescriptor. Pretty straightforward :) Eike, A problem I experienced: I couldn't restore persisted status of the enablement, since Activator can't use/refer to a PREF attribute of OM during activation :( Any sugestion?
Created attachment 121535 [details] patch v2 - incomplete - Fixed problems with IItemLabelProvider. - Added cdoDirectResource
Hi Vik, I looked at the patch and wonder why you do not use the advanced properties to show the informations ? Using private static boolean debugDescriptors = false; isn't very flexible... Do you expect to change that in the future ?
Hi Simon, I just didn't know of the existence of such filter! Is perfect for this feature :) This way I'm able to remove that boolean and the OMPreference (configurable through the preferences page). Much cleaner this way. Thanks! (In reply to comment #19) > Hi Vik, > > I looked at the patch and wonder why you do not use the advanced properties to > show the informations ? > > Using > > private static boolean debugDescriptors = false; > > isn't very flexible... > > Do you expect to change that in the future ? >
Created attachment 121813 [details] patch v3 - Changed methodology: now using advanced properties filter. Removed from preferences page. - All requested features to show are ready: * Container * DirectResource * CDOState * CDOView * ReadLock * WriteLock * OID * Adapters * Subscription types * Revision Version - I overrode toString of the CDOAdapterPolicies instances, so it shows something meaningfull to the user instead of "CDOAdapterPolicies@f82ab12". - I've realized that CDOObjects always show they have a ReadLock! Do I miss something?? Cheers, ViK.
Created attachment 121944 [details] patch v4 - Added descriptor base class for CDO - made descriptor classes public - Added category
Created attachment 121993 [details] Patch v5 - ready to be committed Very well done! It looks lovely ;-) One note though: Certain state changes of an object are not reflected immediately (id, state, locks, etc). Can we become even better?
Re-merged and committed to HEAD
Fix available in CDO 2.0.0M5
Generally available.
