[Rubyeclipse-commit] SF.net SVN: rubyeclipse: [2614] trunk/org.rubypeople.rdt.testunit/src/org/ rubypeople/rdt/internal/testunit/ui/TestUnitView.java

[<caw...@us...>]

Revision: 2614
          http://svn.sourceforge.net/rubyeclipse/?rev=2614&view=rev
Author:   cawilliams
Date:     2007-06-13 11:46:11 -0700 (Wed, 13 Jun 2007)

Log Message:
-----------
Fix #4818 - we were setting the title using toString on an IRubyElement which spits out ugly info. We should just use the fully qualified name if we have a type, or just call getElementName() if it's not a type.

Modified Paths:
--------------
    trunk/org.rubypeople.rdt.testunit/src/org/rubypeople/rdt/internal/testunit/ui/TestUnitView.java

Modified: trunk/org.rubypeople.rdt.testunit/src/org/rubypeople/rdt/internal/testunit/ui/TestUnitView.java
===================================================================
--- trunk/org.rubypeople.rdt.testunit/src/org/rubypeople/rdt/internal/testunit/ui/TestUnitView.java	2007-06-13 13:29:14 UTC (rev 2613)
+++ trunk/org.rubypeople.rdt.testunit/src/org/rubypeople/rdt/internal/testunit/ui/TestUnitView.java	2007-06-13 18:46:11 UTC (rev 2614)
@@ -501,8 +501,8 @@
 		setViewPartTitle(type);
 		if (type instanceof IType)
 			setTitleToolTip(((IType)type).getFullyQualifiedName());
-//		else
-//			setTitleToolTip(type.getElementName());
+		else
+			setTitleToolTip(type.getElementName());
 	}
 
 	protected void aboutToLaunch() {
@@ -596,8 +596,12 @@
 		String title;
 		if (type == null)
 			title = " "; //$NON-NLS-1$
-		else
-			title = type.toString();
+		else {
+			if (type instanceof IType) {
+				title = ((IType) type).getFullyQualifiedName();
+			} else
+				title = type.getElementName();
+		}
 		setContentDescription(title);
 	}
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.