Return-Path: Delivered-To: apmail-directory-commits-archive@www.apache.org Received: (qmail 27719 invoked from network); 27 Feb 2007 16:59:18 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 27 Feb 2007 16:59:18 -0000 Received: (qmail 43029 invoked by uid 500); 27 Feb 2007 16:59:26 -0000 Delivered-To: apmail-directory-commits-archive@directory.apache.org Received: (qmail 42980 invoked by uid 500); 27 Feb 2007 16:59:26 -0000 Mailing-List: contact commits-help@directory.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@directory.apache.org Delivered-To: mailing list commits@directory.apache.org Received: (qmail 42969 invoked by uid 99); 27 Feb 2007 16:59:26 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 Feb 2007 08:59:26 -0800 X-ASF-Spam-Status: No, hits=-99.5 required=10.0 tests=ALL_TRUSTED,NO_REAL_NAME X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO eris.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 Feb 2007 08:59:17 -0800 Received: by eris.apache.org (Postfix, from userid 65534) id 0CCAC1A981F; Tue, 27 Feb 2007 08:58:57 -0800 (PST) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r512323 - in /directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas: controller/ view/viewers/ Date: Tue, 27 Feb 2007 16:58:56 -0000 To: commits@directory.apache.org From: pamarcelot@apache.org X-Mailer: svnmailer-1.1.0 Message-Id: <20070227165857.0CCAC1A981F@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: pamarcelot Date: Tue Feb 27 08:58:55 2007 New Revision: 512323 URL: http://svn.apache.org/viewvc?view=rev&rev=512323 Log: Fixed a bug that caused the TreeViewers to close every opened node when refreshing the viewer. The opened nodes now stay correctly opened. Modified: directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/HierarchyViewController.java directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/SchemasViewController.java directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/HierarchyView.java directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/SchemasView.java Modified: directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/HierarchyViewController.java URL: http://svn.apache.org/viewvc/directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/HierarchyViewController.java?view=diff&rev=512323&r1=512322&r2=512323 ============================================================================== --- directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/HierarchyViewController.java (original) +++ directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/HierarchyViewController.java Tue Feb 27 08:58:55 2007 @@ -229,7 +229,7 @@ { if ( authorizedPrefs.contains( event.getProperty() ) ) { - view.getViewer().refresh(); + view.refresh(); } } } ); Modified: directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/SchemasViewController.java URL: http://svn.apache.org/viewvc/directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/SchemasViewController.java?view=diff&rev=512323&r1=512322&r2=512323 ============================================================================== --- directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/SchemasViewController.java (original) +++ directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/controller/SchemasViewController.java Tue Feb 27 08:58:55 2007 @@ -591,7 +591,7 @@ { if ( authorizedPrefs.contains( event.getProperty() ) ) { - view.getViewer().refresh(); + view.refresh(); } } } ); Modified: directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/HierarchyView.java URL: http://svn.apache.org/viewvc/directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/HierarchyView.java?view=diff&rev=512323&r1=512322&r2=512323 ============================================================================== --- directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/HierarchyView.java (original) +++ directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/HierarchyView.java Tue Feb 27 08:58:55 2007 @@ -94,17 +94,16 @@ /** - * Refreshes the entire view + * Refresh the viewer */ public void refresh() { - //it seems there is a bug with the default element expanding system Object[] exp = viewer.getExpandedElements(); - //refresh the tree viewer + // Refresh the tree viewer viewer.refresh(); - //expand all the previsouly expanded elements + // Expand all the previsouly expanded elements for ( Object object : exp ) { viewer.setExpandedState( object, true ); Modified: directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/SchemasView.java URL: http://svn.apache.org/viewvc/directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/SchemasView.java?view=diff&rev=512323&r1=512322&r2=512323 ============================================================================== --- directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/SchemasView.java (original) +++ directory/ldapstudio/trunk/ldapstudio-schemas-plugin/src/main/java/org/apache/directory/ldapstudio/schemas/view/viewers/SchemasView.java Tue Feb 27 08:58:55 2007 @@ -94,17 +94,16 @@ /** - * Refresh the entire view + * Refresh the viewer */ public void refresh() { - //it seems there is a bug with the default element expanding system Object[] exp = viewer.getExpandedElements(); - //refresh the tree viewer + // Refresh the tree viewer viewer.refresh(); - //expand all the previsouly expanded elements + // Expand all the previsouly expanded elements for ( Object object : exp ) { viewer.setExpandedState( object, true ); @@ -117,8 +116,7 @@ */ public void poolChanged( SchemaPool p, LDAPModelEvent e ) { - //refresh the tree viewer - viewer.refresh(); + refresh(); } @@ -157,20 +155,6 @@ { logger.debug( "error when saving schemas on disk after asking for confirmation" ); //$NON-NLS-1$ } - // - // IWorkbenchPage page = PlatformUI.getWorkbench().getActiveWorkbenchWindow().getActivePage(); - // IEditorPart[] editors = page.getDirtyEditors(); - // for (IEditorPart part : editors) { - // if(part instanceof AttributeTypeFormEditor) { - // AttributeTypeFormEditor editor = (AttributeTypeFormEditor) part; - // editor.setDirty(false); - // } - // else if (part instanceof ObjectClassFormEditor) { - // ObjectClassFormEditor editor = (ObjectClassFormEditor) part; - // editor.setDirty(false); - // } - // } - }