[jira] Created: (OFBIZ-2850) Remove "Various" from @throws javadoc in UelUtil.java

[Nicolas Malin (Jira)]

Remove "Various" from @throws javadoc in UelUtil.java
-----------------------------------------------------

                 Key: OFBIZ-2850
                 URL: https://issues.apache.org/jira/browse/OFBIZ-2850
             Project: OFBiz
          Issue Type: Bug
          Components: framework
    Affects Versions: SVN trunk
            Reporter: Bob Morley
            Priority: Trivial
         Attachments: OFBIZ-2850_RemoveVariousFromJavadoc.patch

Ok this is a trivial little fix; but it did catch me up and require that I commit again and trigger our daily build.

In Eclipse if you hit your "organize imports" on this particular class it interprets the javadoc comment "@throws Various <code>javax.el.*</code> exceptions" and requiring that the "Various" class (org.ofbiz.project.Various) be an import.  In Eclipse all compiles fine (I don't even look how it organized the imports, I just trust that so you don't even see it) but when you run the external ant build you get a classpath dependency problem (framework referencing special purpose).

So this fix simply changes the javadoc comment by taking out the "Various" part.  *whew* :)

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-2850?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Bob Morley updated OFBIZ-2850:
------------------------------

    Attachment: OFBIZ-2850_RemoveVariousFromJavadoc.patch

> Remove "Various" from @throws javadoc in UelUtil.java
> -----------------------------------------------------
>
>                 Key: OFBIZ-2850
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2850
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Priority: Trivial
>         Attachments: OFBIZ-2850_RemoveVariousFromJavadoc.patch
>
>
> Ok this is a trivial little fix; but it did catch me up and require that I commit again and trigger our daily build.
> In Eclipse if you hit your "organize imports" on this particular class it interprets the javadoc comment "@throws Various <code>javax.el.*</code> exceptions" and requiring that the "Various" class (org.ofbiz.project.Various) be an import.  In Eclipse all compiles fine (I don't even look how it organized the imports, I just trust that so you don't even see it) but when you run the external ant build you get a classpath dependency problem (framework referencing special purpose).
> So this fix simply changes the javadoc comment by taking out the "Various" part.  *whew* :)

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-2850?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adrian Crum reassigned OFBIZ-2850:
----------------------------------

    Assignee: Adrian Crum

> Remove "Various" from @throws javadoc in UelUtil.java
> -----------------------------------------------------
>
>                 Key: OFBIZ-2850
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2850
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Assignee: Adrian Crum
>            Priority: Trivial
>         Attachments: OFBIZ-2850_RemoveVariousFromJavadoc.patch
>
>
> Ok this is a trivial little fix; but it did catch me up and require that I commit again and trigger our daily build.
> In Eclipse if you hit your "organize imports" on this particular class it interprets the javadoc comment "@throws Various <code>javax.el.*</code> exceptions" and requiring that the "Various" class (org.ofbiz.project.Various) be an import.  In Eclipse all compiles fine (I don't even look how it organized the imports, I just trust that so you don't even see it) but when you run the external ant build you get a classpath dependency problem (framework referencing special purpose).
> So this fix simply changes the javadoc comment by taking out the "Various" part.  *whew* :)

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

[Nicolas Malin (Jira)]

     [ https://issues.apache.org/jira/browse/OFBIZ-2850?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Adrian Crum closed OFBIZ-2850.
------------------------------

    Resolution: Fixed

Fixed, trunk rev 806377, R 9.04 rev 806378.

Thanks Bob!

> Remove "Various" from @throws javadoc in UelUtil.java
> -----------------------------------------------------
>
>                 Key: OFBIZ-2850
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-2850
>             Project: OFBiz
>          Issue Type: Bug
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Bob Morley
>            Assignee: Adrian Crum
>            Priority: Trivial
>         Attachments: OFBIZ-2850_RemoveVariousFromJavadoc.patch
>
>
> Ok this is a trivial little fix; but it did catch me up and require that I commit again and trigger our daily build.
> In Eclipse if you hit your "organize imports" on this particular class it interprets the javadoc comment "@throws Various <code>javax.el.*</code> exceptions" and requiring that the "Various" class (org.ofbiz.project.Various) be an import.  In Eclipse all compiles fine (I don't even look how it organized the imports, I just trust that so you don't even see it) but when you run the external ant build you get a classpath dependency problem (framework referencing special purpose).
> So this fix simply changes the javadoc comment by taking out the "Various" part.  *whew* :)

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.