[jira] [Commented] (ACE-201) Can't change a target name/description

["Denis Koelewijn (Commented) (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ACE-201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13140476#comment-13140476 ] 

Denis Koelewijn commented on ACE-201:
-------------------------------------

Is this new behaviour ?

>From the sourcecode it looks like this is intended behaviour. It is allowed for Artifacts to have an (editable) description, but not for Targets. This hasn't been modified recently.

If indeed it should be impossible to edit the description field, then I think it would be better to remove this field altogether from the edit window for targets 

On the other hand, giving the operator the option to add an extra description to a target would probably make sense as well. This latter option is however not directly mappable to the underlying objects.
                
> Can't change a target name/description
> --------------------------------------
>
>                 Key: ACE-201
>                 URL: https://issues.apache.org/jira/browse/ACE-201
>             Project: Ace
>          Issue Type: Bug
>          Components: Web UI
>            Reporter: Jean-Baptiste Onofré
>
> When we try to change a target name or description, we get:
> SEVERE: Terminal error:
> com.vaadin.event.ListenerMethod$MethodException
> Cause: java.lang.IllegalArgumentException
>         at com.vaadin.event.ListenerMethod.receiveEvent(ListenerMethod.java:510)
>         at com.vaadin.event.EventRouter.fireEvent(EventRouter.java:162)
>         at com.vaadin.ui.AbstractComponent.fireEvent(AbstractComponent.java:1166)
>         at com.vaadin.ui.Button.fireClick(Button.java:380)
>         at com.vaadin.ui.Button.changeVariables(Button.java:196)
>         at com.vaadin.terminal.gwt.server.AbstractCommunicationManager.handleVariableBurst(AbstractCommunicationManager.java:1297)
>         at com.vaadin.terminal.gwt.server.AbstractCommunicationManager.handleVariables(AbstractCommunicationManager.java:1217)
>         at com.vaadin.terminal.gwt.server.AbstractCommunicationManager.doHandleUidlRequest(AbstractCommunicationManager.java:733)
>         at com.vaadin.terminal.gwt.server.CommunicationManager.handleUidlRequest(CommunicationManager.java:296)
>         at com.vaadin.terminal.gwt.server.AbstractApplicationServlet.service(AbstractApplicationServlet.java:483)
>         at javax.servlet.http.HttpServlet.service(HttpServlet.java:820)
>         at org.mortbay.jetty.servlet.ServletHolder.handle(ServletHolder.java:502)
>         at org.mortbay.jetty.servlet.ServletHandler.handle(ServletHandler.java:389)
>         at org.ops4j.pax.web.service.jetty.internal.HttpServiceServletHandler.handle(HttpServiceServletHandler.java:64)
>         at org.mortbay.jetty.servlet.SessionHandler.handle(SessionHandler.java:181)
>         at org.mortbay.jetty.handler.ContextHandler.handle(ContextHandler.java:765)
>         at org.ops4j.pax.web.service.jetty.internal.HttpServiceContext.handle(HttpServiceContext.java:111)
>         at org.ops4j.pax.web.service.jetty.internal.JettyServerHandlerCollection.handle(JettyServerHandlerCollection.java:64)
>         at org.mortbay.jetty.handler.HandlerWrapper.handle(HandlerWrapper.java:152)
>         at org.mortbay.jetty.Server.handle(Server.java:326)
>         at org.mortbay.jetty.HttpConnection.handleRequest(HttpConnection.java:534)
>         at org.mortbay.jetty.HttpConnection$RequestHandler.content(HttpConnection.java:879)
>         at org.mortbay.jetty.HttpParser.parseNext(HttpParser.java:747)
>         at org.mortbay.jetty.HttpParser.parseAvailable(HttpParser.java:218)
>         at org.mortbay.jetty.HttpConnection.handle(HttpConnection.java:404)
>         at org.mortbay.io.nio.SelectChannelEndPoint.run(SelectChannelEndPoint.java:409)
>         at org.mortbay.thread.QueuedThreadPool$PoolThread.run(QueuedThreadPool.java:520)
> Caused by: java.lang.IllegalArgumentException
>         at org.apache.ace.webui.vaadin.NamedTargetObject.setDescription(NamedTargetObject.java:34)
>         at org.apache.ace.webui.vaadin.EditWindow$1.buttonClick(EditWindow.java:74)
>         at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>         at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
>         at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
>         at java.lang.reflect.Method.invoke(Method.java:597)
>         at com.vaadin.event.ListenerMethod.receiveEvent(ListenerMethod.java:490)
>         ... 26 more

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira