Bug 832066 - Helloworld Errai: javascript validation error in JBDS
Summary: Helloworld Errai: javascript validation error in JBDS
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise WFK Platform 2
Classification: Retired
Component: Errai
Version: 2.0.0.ER8
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ER1
: 2.2.0
Assignee: Pavel SLegr
QA Contact: Tomas Repel
URL:
Whiteboard: jboss-as-quickstarts jboss-as-hellowo...
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-06-14 12:31 UTC by Tomas Repel
Modified: 2013-05-14 14:54 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Invalid JavaScript files were generated by GWT when packaging the helloworld-errai quickstart application using Maven. After importing the packaged application into JBoss Developer Studio, errors were thrown about the invalid JavaScript files. The issue has been fixed in this release and the application can now be packaged and imported without throwing any errors.
Clone Of:
Environment:
Last Closed: 2013-05-14 14:54:47 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 828951 0 urgent CLOSED Add WFK specific quickstarts 2021-02-22 00:41:40 UTC
Red Hat Bugzilla 874090 0 unspecified CLOSED errai-bus examples - java script validation error in JBDS for various projects 2021-02-22 00:41:40 UTC

Internal Links: 828951 874090

Description Tomas Repel 2012-06-14 12:31:11 UTC
Description of problem:

After importing Helloworld Errai quickstart, the javascript validation error occurs.

Version-Release number of selected component (if applicable):

WFK2 ER8
EAP6 CR1
JBDS5 CR1

How reproducible:

Always.

Steps to Reproduce:
1. import jboss-as-quickstart-helloworld-errai into JBDS
  
Actual results:

Javascript validation error occurs.

Expected results:

No errors should be shown after import.

Additional info:

The workaround is to exclude specific JS files (5.cache.js) from being validated.

Comment 1 Karel Piwko 2012-06-20 15:05:29 UTC
    Technical note added. If any revisions are required, please edit the "Technical Notes" field
    accordingly. All revisions will be proofread by the Engineering Content Services team.
    
    New Contents:
JBoss Developer Studio complains about JavaScript files distributed with the example being invalid. This does not impair functionality of the example.

To get rid of the error, exclude specific JS files (5.cache.js) from being validated.

Comment 6 Jonathan Fuerth 2012-09-27 14:09:09 UTC
I'd recommend disabling JavaScript validation on everything under src/main/webapp/${moduleName}/**.

Rationale:

1. The warnings that we get from JavaScript validation are pointing out potential programming problems (that is, the JavaScript code may not be doing what the author intended). Although this is really useful for handwritten code, these warnings are always false positives with code generated by GWT. The code does exactly what the GWT compiler intended.

2. JavaScript validation takes time to complete, so even if people learn to ignore the warnings, their IDE will be hammering the CPU needlessly.

Comment 7 Marek Novotny 2012-10-01 11:58:07 UTC
Thanks, Jonathan!

Karel or Tomas, we need to change a little Release notes text to say to exclude that directory from JS validations

Comment 8 Karel Piwko 2012-10-03 15:36:10 UTC
Marek, can't we rather ship JBDS configuration disabling the JS validation with the example out of the box?

Comment 9 Marek Novotny 2012-10-19 12:27:37 UTC
agree that we should provide everything what user/customer can find easy to use, but I have some concerns with mixing really wanted eclipse settings and by mistake added other settings from local developments.

Comment 10 mark yarborough 2012-11-05 14:51:40 UTC
Has been moved 2 releases, now to WFK 2.2. Is a "known issue" with a simple workaround with JBDS. Decide before WFK 2.2 whether is "wont fix" or if we can address in WFK...

Comment 11 Pavel SLegr 2013-03-05 09:19:01 UTC
I can not reproduce this issue anymore:
(this is related to bug 874090 - Stefan already verified)

Testing with:
WFK-2.2.0.ER1
EAP-6.0.1.GA
JBDS:
Version: 6.0.0.GA
Build id: GA-v20121206-1855-B186
Build date: 20121206-1855

Please confirm this issue is gone

Comment 12 Tomas Repel 2013-03-05 09:41:05 UTC
True, using exactly the same versions as Pavel gives no errors after importing. Changing status to verified.

Comment 13 Karel Piwko 2013-05-14 14:54:47 UTC
Closed as CURRENTRELEASE, part of WFK 2.2.0 distributables.


Note You need to log in before you can comment on or make changes to this bug.