Re: [gwt-contrib] Re: Add README step to use the Google Web Toolkit SDK in the gwt-user project so that users can run ... (issue966802)

2010-11-27 Thread Miguel Méndez
Actually, I'm not sure that this is accurate.  I just tried it out against
gwt-user and gwt-dev with natures and builders on in a clean eclipse Helios
install (no GPE) and I did not see any exceptions thrown.  There were no
visible exceptions and the error log was not updated.

It seems worth it to take a look and see if it is eclipse version specific.

On Wed, Nov 24, 2010 at 10:31 AM, rda...@google.com wrote:

 I think that jat is right - if you have a builder defined on your
 project that Eclipse does not know about, it will throw an exception.

 For now, let's go with this update to the README. I think, as a future
 step, we should add the natures and builders to the gwt-user and gwt-dev
 project, and make it a necessary step for them to install GPE.


 http://gwt-code-reviews.appspot.com/966802/show

 --
 http://groups.google.com/group/Google-Web-Toolkit-Contributors




-- 
Miguel

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

[gwt-contrib] Issue 5474: RequestFactory response encoding (issue1154801)

2010-11-27 Thread t . broyer

Reviewers: bobv,

Description:
Fix for RequestFactoryServlet and unit test.

Also fixed testUserInfo in RequestFactoryTest, which was not
asynchronous (and therefore, I believe, wasn't actually testing
anything).

The tests use finishTest and not finishTestAndReset because they don't
use SimpleFooRequest and SimpleBarRequest, so they don't need the extra
reset() requests.

Please review this at http://gwt-code-reviews.appspot.com/1154801/show

Affected files:
  user/src/com/google/gwt/requestfactory/server/RequestFactoryServlet.java
  user/test/com/google/gwt/requestfactory/client/RequestFactoryTest.java
  user/test/com/google/gwt/requestfactory/server/Simple.java
  user/test/com/google/gwt/requestfactory/shared/SimpleRequest.java
  user/test/com/google/gwt/requestfactory/shared/SimpleRequestFactory.java


--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


[gwt-contrib] Re: Comment on OverlayTypes in google-web-toolkit

2010-11-27 Thread codesite-noreply

Comment by ltearno:

This is too bad (at the end of the document) :

The implementation of overlay types in GWT 1.5 does not support Generators  
defining new JSO subtypes (due to the need to redefine or otherwise extend  
JavaScriptObject$ during subsequent compilation). This restriction is still  
in place.



Being able to generate sub classes of JavaScriptObject would be so coool

For more information:
http://code.google.com/p/google-web-toolkit/wiki/OverlayTypes

--
http://groups.google.com/group/Google-Web-Toolkit-Contributors


Re: [gwt-contrib] Re: r9277 committed - Removing reference to designTime.getProvidedField()

2010-11-27 Thread Stephanie Brubaker
Konstantin, we have a separate release branch for 2.1.1 that is forked from
trunk.  Your change has not been integrated into that branch, so when I
picked up a piece of your code in UiBinderWriter it broke testing in the
release branch. This change was only applied to the release branch, not to
trunk, so your code should be intact.  Please let me know if you're seeing
any issues with trunk.

Thanks,
Stephanie

On Thu, Nov 25, 2010 at 3:22 PM, Konstantin.Scheglov 
konstantin.scheg...@gmail.com wrote:



  Then other pieces of the change were not integrated, as this was
 preventing
  the 2.1 branch from building.

   I see method getProvidedField() in DesignTimeUtils in SVN trunk, so
 I'm not sure what could be missed.
  I also regularly build GWT from project in SVN without build
 problems.


  On Nov 25, 2010 8:12 AM, Konstantin.Scheglov 
 
 
 
 
 
 
 
  konstantin.scheg...@gmail.com wrote:
 
   No!
   This change breaks @UiField(provided=true) support in GWT Designer,
   important feature which we want to support for GWT 2.1.1.
 
   On 24 ноя, 19:55, codesite-nore...@google.com wrote:
   Revision: 9277
   Author: gwt.mirror...@gmail.com
   Date: Wed Nov 24 08:54:59 2010
   Log: Removing reference to designTime.getProvidedField()
 
  http://code.google.com/p/google-web-toolkit/source/detail?r=9277
 
   Modified:
 
  /releases/2.1/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java
 
 
 
 
 
 
 
 
 
   ===
   ---
  
 /releases/2.1/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java
   Fri Nov 19 05:49:25 2010
   +++
  
 /releases/2.1/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java
   Wed Nov 24 08:54:59 2010
   @@ -1093,15 +1093,8 @@
 
 // TODO why can this be null?
 if (fieldWriter != null) {
   -  String initializer;
   -  if (designTime.isDesignTime()) {
   -String typeName =
   ownerField.getType().getRawType().getQualifiedSourceName();
   -initializer = designTime.getProvidedField(typeName,
   -ownerField.getName());
   -  } else {
   -initializer = formatCode(owner.%1$s, fieldName);
   -  }
   -  fieldManager.lookup(fieldName).setInitializer(initializer);
   +  fieldManager.lookup(fieldName).setInitializer(
   +  formatCode(owner.%1$s, fieldName));
 }
   }
 }
   @@ -1215,4 +1208,3 @@
 designTime.addDeclarations(w);
   }
 }
   -
 
   --
  http://groups.google.com/group/Google-Web-Toolkit-Contributors

 --
 http://groups.google.com/group/Google-Web-Toolkit-Contributors


-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Re: [gwt-contrib] Re: r9277 committed - Removing reference to designTime.getProvidedField()

2010-11-27 Thread Konstantin Scheglov
Konstantin, we have a separate release branch for 2.1.1 that is forked from
 trunk.  Your change has not been integrated into that branch, so when I
 picked up a piece of your code in UiBinderWriter it broke testing in the
 release branch. This change was only applied to the release branch, not to
 trunk, so your code should be intact.  Please let me know if you're seeing
 any issues with trunk.


  Current state in trunk is valid.

  Will final version of GWT 2.1.1 which users will download still have
design time tweaks of UiBinderWriter?





 Thanks,
 Stephanie


 On Thu, Nov 25, 2010 at 3:22 PM, Konstantin.Scheglov 
 konstantin.scheg...@gmail.com wrote:



  Then other pieces of the change were not integrated, as this was
 preventing
  the 2.1 branch from building.

   I see method getProvidedField() in DesignTimeUtils in SVN trunk, so
 I'm not sure what could be missed.
  I also regularly build GWT from project in SVN without build
 problems.


  On Nov 25, 2010 8:12 AM, Konstantin.Scheglov 
 
 
 
 
 
 
 
  konstantin.scheg...@gmail.com wrote:
 
   No!
   This change breaks @UiField(provided=true) support in GWT Designer,
   important feature which we want to support for GWT 2.1.1.
 
   On 24 ноя, 19:55, codesite-nore...@google.com wrote:
   Revision: 9277
   Author: gwt.mirror...@gmail.com
   Date: Wed Nov 24 08:54:59 2010
   Log: Removing reference to designTime.getProvidedField()
 
  http://code.google.com/p/google-web-toolkit/source/detail?r=9277
 
   Modified:
 
 
 /releases/2.1/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java
 
 
 
 
 
 
 
 
 
   ===
   ---
  
 /releases/2.1/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java
   Fri Nov 19 05:49:25 2010
   +++
  
 /releases/2.1/user/src/com/google/gwt/uibinder/rebind/UiBinderWriter.java
   Wed Nov 24 08:54:59 2010
   @@ -1093,15 +1093,8 @@
 
 // TODO why can this be null?
 if (fieldWriter != null) {
   -  String initializer;
   -  if (designTime.isDesignTime()) {
   -String typeName =
   ownerField.getType().getRawType().getQualifiedSourceName();
   -initializer = designTime.getProvidedField(typeName,
   -ownerField.getName());
   -  } else {
   -initializer = formatCode(owner.%1$s, fieldName);
   -  }
   -
  fieldManager.lookup(fieldName).setInitializer(initializer);
   +  fieldManager.lookup(fieldName).setInitializer(
   +  formatCode(owner.%1$s, fieldName));
 }
   }
 }
   @@ -1215,4 +1208,3 @@
 designTime.addDeclarations(w);
   }
 }
   -
 
   --
  http://groups.google.com/group/Google-Web-Toolkit-Contributors

 --
 http://groups.google.com/group/Google-Web-Toolkit-Contributors


  --
 http://groups.google.com/group/Google-Web-Toolkit-Contributors




-- 
Konstantin Scheglov,
Google, Inc.

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors