You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@activemq.apache.org by "Timothy Bish (JIRA)" <ji...@apache.org> on 2009/09/25 21:00:52 UTC

[jira] Resolved: (AMQCPP-262) exception message in checkWriteOnlyBody is wrong

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

Timothy Bish resolved AMQCPP-262.
---------------------------------

       Resolution: Fixed
    Fix Version/s: 3.1

Patch applied, thanks!

> exception message in checkWriteOnlyBody is wrong
> ------------------------------------------------
>
>                 Key: AMQCPP-262
>                 URL: https://issues.apache.org/activemq/browse/AMQCPP-262
>             Project: ActiveMQ C++ Client
>          Issue Type: Bug
>          Components: CMS Impl
>            Reporter: Eric van Gyzen
>            Assignee: Timothy Bish
>            Priority: Trivial
>             Fix For: 3.1
>
>         Attachments: checkWriteOnlyBody.diff
>
>
> In the checkWriteOnlyBody method of the ActiveMQBytesMessage and ActiveMQStreamMessage classes, the message in the exception claims the opposite of reality.
> Index: activemq-cpp/src/main/activemq/commands/ActiveMQBytesMessage.cpp
> ===================================================================
> --- activemq-cpp/src/main/activemq/commands/ActiveMQBytesMessage.cpp    (revision 817354)
> +++ activemq-cpp/src/main/activemq/commands/ActiveMQBytesMessage.cpp    (working copy)
> @@ -70,8 +70,8 @@
>      if( !this->isReadOnlyBody() ){
>          throw exceptions::ActiveMQException(
>              __FILE__, __LINE__,
> -            "message is in read-only mode and "
> -            "cannot be written to" ).convertToCMSException();
> +            "message is in write-only mode and "
> +            "cannot be read from" ).convertToCMSException();
>      }
>  }
> Index: activemq-cpp/src/main/activemq/commands/ActiveMQStreamMessage.cpp
> ===================================================================
> --- activemq-cpp/src/main/activemq/commands/ActiveMQStreamMessage.cpp   (revision 817354)
> +++ activemq-cpp/src/main/activemq/commands/ActiveMQStreamMessage.cpp   (working copy)
> @@ -90,8 +90,8 @@
>      if( !this->isReadOnlyBody() ){
>          throw exceptions::ActiveMQException(
>              __FILE__, __LINE__,
> -            "message is in read-only mode and "
> -            "cannot be written to" ).convertToCMSException();
> +            "message is in write-only mode and "
> +            "cannot be read from" ).convertToCMSException();
>      }
>  }

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