Support for generics return types in Async interface generation

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.1
  • Labels:
    None

Description

Based on this method in the sync GWT-RPC interface :

List<Numero> getListeNumeros();

the generated async interface is :

void getListeNumeros( AsyncCallback<List> callback );

It may be :

void getListeNumeros( AsyncCallback<List<Numero>> callback );

  1. Text File
    GWT-1.patch
    02/Jan/09 1:33 PM
    8 kB
    Robert Scholte
  2. Text File
    MGWT-1.1
    02/Feb/09 10:53 AM
    0.4 kB
    Robert Scholte

Issue Links

depends upon

New Feature - A new feature of the product, which has yet to be developed. QDOX-54 Support for retrieval of generic type information (JSR 14)

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Robert Scholte added a comment -

fix in qdox is available in 1.8-SNAPSHOT. With this fix this issue can be resolved. Modified code and a test added to this patch

Show
Robert Scholte added a comment - fix in qdox is available in 1.8-SNAPSHOT. With this fix this issue can be resolved. Modified code and a test added to this patch
Hide
Permalink
Olivier Lamy added a comment -

Thanks for the patch.
But we will depend on a qdox release.
Any idea when it's planned ?

Show
Olivier Lamy added a comment - Thanks for the patch. But we will depend on a qdox release. Any idea when it's planned ?
Hide
Permalink
Robert Scholte added a comment -

They've just release 1.7 on december 22, so we've just missed that one. Although this is a very interesting fix in qdox I don't expect a new release very soon, unless we can convince them to plan a new release pretty soon (what about suggesting a 1.7.1).
I think it's more convincing to contact the developers of qdox yourself and mention the importance of this dependency for a new gwt-plugin release.

Show
Robert Scholte added a comment - They've just release 1.7 on december 22, so we've just missed that one. Although this is a very interesting fix in qdox I don't expect a new release very soon, unless we can convince them to plan a new release pretty soon (what about suggesting a 1.7.1). I think it's more convincing to contact the developers of qdox yourself and mention the importance of this dependency for a new gwt-plugin release.
Hide
Permalink
Olivier Lamy added a comment -

fixed in rev 8660
Thanks.

Show
Olivier Lamy added a comment - fixed in rev 8660 Thanks.
Hide
Permalink
nicolas de loof added a comment -

Can we wait for a QDox release to close this issue ?
I'd like to avoid a release to be blocked by such SNAPSHOT dependencies.

Show
nicolas de loof added a comment - Can we wait for a QDox release to close this issue ? I'd like to avoid a release to be blocked by such SNAPSHOT dependencies.
Hide
Permalink
Olivier Lamy added a comment -

I reopen it as it's not really fixed because we depend on a qdox SNAPSHOT.
If we want to release gwt plugin the commit will be reverted if qdox has not been released.

Show
Olivier Lamy added a comment - I reopen it as it's not really fixed because we depend on a qdox SNAPSHOT. If we want to release gwt plugin the commit will be reverted if qdox has not been released.
Hide
Permalink
Robert Scholte added a comment -

qdox-1.8 has just been released. No more SNAPSHOT-version required

Show
Robert Scholte added a comment - qdox-1.8 has just been released. No more SNAPSHOT-version required
Hide
Permalink
Olivier Lamy added a comment -

use qdox released version in rev 8972.

Show
Olivier Lamy added a comment - use qdox released version in rev 8972.
Hide
Permalink
Jeff MAURY added a comment -

I'm having the problem with the 2.1.0 release of the plugin which is using qdox 1.12. Regression ?

Show
Jeff MAURY added a comment - I'm having the problem with the 2.1.0 release of the plugin which is using qdox 1.12. Regression ?
Hide
Permalink
Olivier Lamy added a comment -

weird because there is an it test to test async stuff.
If you have a simple project to reproduce your issue.
Please open an other issue with your sample.
Thanks !

Show
Olivier Lamy added a comment - weird because there is an it test to test async stuff. If you have a simple project to reproduce your issue. Please open an other issue with your sample. Thanks !
Hide
Permalink
Jeff MAURY added a comment -

With a generated sample, it works. It seems related to interaction with M2Eclipse as, after running mvn clean on my project, it came back ok

Show
Jeff MAURY added a comment - With a generated sample, it works. It seems related to interaction with M2Eclipse as, after running mvn clean on my project, it came back ok

People

  • Assignee:
    Olivier Lamy
    Reporter:
    nicolas de loof
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: