[forge-dev] UIValidator

Ivan St. Ivanov ivan.st.ivanov at gmail.com
Sat Jan 3 08:21:27 EST 2015


Yeah, we may leave the IllegalArgumentException and just add another catch
clause for RuntimeException

On Sat, Jan 3, 2015 at 3:14 PM, George Gastaldi <ggastald at redhat.com> wrote:

> That is a good idea, but I guess also that we should not wrap the
> IllegalArgumentException thrown in MavenAddonDependencyResolver.
>
> Best Regards,
>
> George Gastaldi
>
> Em 03/01/2015, às 11:09, Ivan St. Ivanov <ivan.st.ivanov at gmail.com>
> escreveu:
>
> Hi folks,
>
> George, we are talking here about Soro's issue, the one that you reported
> in the JIRA. About mine, you were right that the environment variable was
> not set.
>
> I guess that the validator *is* called. At least from the stack trace in
> the JIRA, I see that the error happens while calling the AbstractUICommand
> validate method. Most probably the exception escapes and just gets logged.
>
> Maybe if you add catch clause for a RuntimeException to the validator, it
> will work.
>
> Cheers,
> Ivan
>
> On Sat, Jan 3, 2015 at 3:03 PM, George Gastaldi <ggastald at redhat.com>
> wrote:
>
>> Hi Ivan,
>>
>> That's because the code is on the Bootstrap class (see the stacktrace you
>> pasted). As for the UIValidator not being called, I'll investigate further.
>>
>>
>>
>> Em 03/01/2015, às 10:51, Ivan St. Ivanov <ivan.st.ivanov at gmail.com>
>> escreveu:
>>
>> Hi folks,
>>
>> *George*, most probably that was the reason.
>>
>> I don't quite understand the problem described in this issue. Is it
>> because there is a validation error but there is no popup in the IDE and
>> just something in the logs? Then I guess this could be fixed if the
>> exception that is caught here
>> <https://github.com/forge/core/blob/master/addon-manager/impl/src/main/java/org/jboss/forge/addon/manager/impl/ui/AddonInstallCommand.java#L98>
>> is changed to RuntimeException or if we add a new catch block for
>> RuntimeException with different validation error.
>>
>> WDYT?
>>
>> Cheers,
>> Ivan
>>
>> On Sat, Jan 3, 2015 at 2:36 PM, Daniel Cunha <danielsoro at gmail.com>
>> wrote:
>>
>>> Hi Ivan,
>>>
>>> I'm looking for it. George opened a issue for it. (
>>> https://issues.jboss.org/browse/FORGE-2169).
>>> http://maven.apache.org/pom.html#Maven_Coordinates
>>>
>>> I think that this "," (comma) not should to exist. Ok?
>>>
>>> On Sat, Jan 3, 2015 at 12:28 PM, Ivan St. Ivanov <
>>> ivan.st.ivanov at gmail.com> wrote:
>>>
>>>> Hi Soro,
>>>>
>>>> I guess upon executing the addon install command (prior to the actual
>>>> execution).
>>>>
>>>> Did you also get an error when trying to run:
>>>>
>>>> forge --install org.jboss.forge.addon:core,$FORGE_VERSION
>>>>
>>>> This morning I got:
>>>>
>>>> java.lang.IllegalArgumentException: Coordinates must be of the form
>>>> 'name,version' or 'name,version,api-version
>>>> at
>>>> org.jboss.forge.furnace.addons.AddonId.fromCoordinates(AddonId.java:78)
>>>> at org.jboss.forge.bootstrap.Bootstrap.install(Bootstrap.java:311)
>>>> at org.jboss.forge.bootstrap.Bootstrap.<init>(Bootstrap.java:179)
>>>> at org.jboss.forge.bootstrap.Bootstrap.main(Bootstrap.java:95)
>>>>
>>>> However, I did not investigate much into it. I guess I have to change
>>>> the `:core` part in my script to `,core`.
>>>>
>>>> Cheers,
>>>> Ivan
>>>>
>>>> On Sat, Jan 3, 2015 at 2:19 PM, Daniel Cunha <danielsoro at gmail.com>
>>>> wrote:
>>>>
>>>>> When this validation is called?
>>>>>
>>>>> https://github.com/forge/core/blob/master/addon-manager/impl/src/main/java/org/jboss/forge/addon/manager/impl/ui/AddonInstallCommand.java#L88-L107
>>>>>
>>>>> --
>>>>> Daniel Cunha (soro)
>>>>>
>>>>> _______________________________________________
>>>>> forge-dev mailing list
>>>>> forge-dev at lists.jboss.org
>>>>> https://lists.jboss.org/mailman/listinfo/forge-dev
>>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> forge-dev mailing list
>>>> forge-dev at lists.jboss.org
>>>> https://lists.jboss.org/mailman/listinfo/forge-dev
>>>>
>>>
>>>
>>>
>>> --
>>> Daniel Cunha (soro)
>>>
>>> _______________________________________________
>>> forge-dev mailing list
>>> forge-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/forge-dev
>>>
>>
>> _______________________________________________
>> forge-dev mailing list
>> forge-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/forge-dev
>>
>>
>> _______________________________________________
>> forge-dev mailing list
>> forge-dev at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/forge-dev
>>
>
> _______________________________________________
> forge-dev mailing list
> forge-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
>
>
> _______________________________________________
> forge-dev mailing list
> forge-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/forge-dev/attachments/20150103/e8c5452d/attachment-0001.html 


More information about the forge-dev mailing list