This is an automated email from the ASF dual-hosted git repository. aharui pushed a commit to branch develop in repository https://gitbox.apache.org/repos/asf/royale-asjs.git
commit c6c6258ee2f3ffc3bce1192e550782e1d6e209ff Author: Alex Harui <aha...@apache.org> AuthorDate: Tue Jan 8 23:20:39 2019 -0800 get rid of warnings in messaging classes --- .../messaging/errors/MessageSerializationError.as | 11 +- .../royale/mx/messaging/messages/CommandMessage.as | 11 +- .../royale/mx/messaging/messages/ErrorMessage.as | 55 ++++++++- .../mx/messaging/messages/HTTPRequestMessage.as | 44 +++++++- .../messaging/messages/MessagePerformanceInfo.as | 123 +++++++++++++++++++-- .../mx/messaging/messages/RemotingMessage.as | 22 +++- 6 files changed, 241 insertions(+), 25 deletions(-) diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/errors/MessageSerializationError.as b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/errors/MessageSerializationError.as index 25e6242..fa4c575 100644 --- a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/errors/MessageSerializationError.as +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/errors/MessageSerializationError.as @@ -67,6 +67,8 @@ package mx.messaging.errors // //-------------------------------------------------------------------------- + private var _fault:ErrorMessage; + /** * Provides specific information about the fault that occurred and for * which message. @@ -77,7 +79,14 @@ package mx.messaging.errors * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var fault:ErrorMessage; + public function get fault():ErrorMessage + { + return _fault; + } + public function set fault(value:ErrorMessage):void + { + _fault = value; + } } } diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/CommandMessage.as b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/CommandMessage.as index 6f79f4b..83498b9 100644 --- a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/CommandMessage.as +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/CommandMessage.as @@ -397,6 +397,8 @@ public class CommandMessage extends AsyncMessage // //-------------------------------------------------------------------------- + private var _operation:uint; + /** * Provides access to the operation/command for the CommandMessage. * Operations indicate how this message should be processed by the remote @@ -408,7 +410,14 @@ public class CommandMessage extends AsyncMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var operation:uint; + public function get operation():uint + { + return _operation; + } + public function set operation(value:uint):void + { + _operation = value; + } //-------------------------------------------------------------------------- // diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/ErrorMessage.as b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/ErrorMessage.as index 76569f4..9e346e3 100644 --- a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/ErrorMessage.as +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/ErrorMessage.as @@ -92,6 +92,8 @@ public class ErrorMessage extends AcknowledgeMessage // //-------------------------------------------------------------------------- + private var _faultCode:String; + /** * The fault code for the error. * This value typically follows the convention of @@ -104,8 +106,17 @@ public class ErrorMessage extends AcknowledgeMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var faultCode:String; + public function get faultCode():String + { + return _faultCode; + } + public function set faultCode(value:String):void + { + _faultCode = value; + } + private var _faultString:String; + /** * A simple description of the error. * @@ -115,8 +126,17 @@ public class ErrorMessage extends AcknowledgeMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var faultString:String; + public function get faultString():String + { + return _faultString; + } + public function set faultString(value:String):void + { + _faultString = value; + } + private var _faultDetail:String; + /** * Detailed description of what caused the error. * This is typically a stack trace from the remote destination. @@ -127,8 +147,17 @@ public class ErrorMessage extends AcknowledgeMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var faultDetail:String; + public function get faultDetail():String + { + return _faultDetail; + } + public function set faultDetail(value:String):void + { + _faultDetail = value; + } + private var _rootCause:Object; + /** * Should a root cause exist for the error, this property contains those details. * This may be an ErrorMessage, a NetStatusEvent info Object, or an underlying @@ -140,7 +169,16 @@ public class ErrorMessage extends AcknowledgeMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var rootCause:Object; + public function get rootCause():Object + { + return _rootCause; + } + public function set rootCause(value:Object):void + { + _rootCause = value; + } + + private var _extendedData:Object; /** * Extended data that the remote destination has chosen to associate @@ -152,7 +190,14 @@ public class ErrorMessage extends AcknowledgeMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var extendedData:Object; + public function get extendedData():Object + { + return _extendedData; + } + public function set extendedData(value:Object):void + { + _extendedData = value; + } //-------------------------------------------------------------------------- diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/HTTPRequestMessage.as b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/HTTPRequestMessage.as index 0660892..bb1d4a6 100644 --- a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/HTTPRequestMessage.as +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/HTTPRequestMessage.as @@ -67,6 +67,8 @@ public class HTTPRequestMessage extends AbstractMessage // //-------------------------------------------------------------------------- + private var _contentType:String; + /** * Indicates the content type of this message. * This value must be understood by the destination this request is sent to. @@ -85,8 +87,17 @@ public class HTTPRequestMessage extends AbstractMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var contentType:String; + public function get contentType():String + { + return _contentType; + } + public function set contentType(value:String):void + { + _contentType = value; + } + private var _httpHeaders:Object; + /** * Contains specific HTTP headers that should be placed on the request made * to the destination. @@ -97,7 +108,16 @@ public class HTTPRequestMessage extends AbstractMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var httpHeaders:Object; + public function get httpHeaders():Object + { + return _httpHeaders; + } + public function set httpHeaders(value:Object):void + { + _httpHeaders = value; + } + + private var _recordHeaders:Boolean; /** * Only used when going through the proxy, should the proxy @@ -110,7 +130,16 @@ public class HTTPRequestMessage extends AbstractMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var recordHeaders:Boolean; + public function get recordHeaders():Boolean + { + return _recordHeaders; + } + public function set recordHeaders(value:Boolean):void + { + _recordHeaders = value; + } + + private var _url:String; [Inspectable(defaultValue="undefined", category="General")] /** @@ -132,7 +161,14 @@ public class HTTPRequestMessage extends AbstractMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var url:String; + public function get url():String + { + return _url; + } + public function set url(value:String):void + { + _url = value; + } /** * @private diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/MessagePerformanceInfo.as b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/MessagePerformanceInfo.as index 06eb5b0..8976d17 100644 --- a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/MessagePerformanceInfo.as +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/MessagePerformanceInfo.as @@ -54,6 +54,8 @@ package mx.messaging.messages // //-------------------------------------------------------------------------- + private var _messageSize:int; + /** * Size of message in Bytes (message types depends on what header this MPI is in) * @@ -63,8 +65,17 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var messageSize:int; + public function get messageSize():int + { + return _messageSize; + } + public function set messageSize(value:int):void + { + _messageSize = value; + } + private var _sendTime:Number = 0; + /** * Millisecond timestamp of when this message was sent * (origin depends on on what header this MPI is in) @@ -75,7 +86,14 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var sendTime:Number = 0; + public function get sendTime():Number + { + return _sendTime; + } + public function set sendTime(value:Number):void + { + _sendTime = value; + } /** * Millisecond timestamp of when this message was received @@ -89,6 +107,8 @@ package mx.messaging.messages */ private var _receiveTime:Number; + private var _overheadTime:Number; + /** * Amount of time in milliseconds that this message was being processed on the server * in order to calculate and populate MPI metrics @@ -99,7 +119,14 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var overheadTime:Number; + public function get overheadTime():Number + { + return _overheadTime; + } + public function set overheadTime(value:Number):void + { + _overheadTime = value; + } /** * "OUT" when this message originated on the server @@ -112,6 +139,8 @@ package mx.messaging.messages */ private var _infoType:String; + private var _pushedFlag:Boolean; + /** * True if this is info for a message that was pushed from server to client * @@ -121,8 +150,17 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var pushedFlag:Boolean; + public function get pushedFlag():Boolean + { + return _pushedFlag; + } + public function set pushedFlag(value:Boolean):void + { + _pushedFlag = value; + } + private var _serverPrePushTime:Number; + /** * Millisecond timestamp of when the server became ready to push this message out * to clients @@ -133,8 +171,17 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var serverPrePushTime:Number; + public function get serverPrePushTime():Number + { + return _serverPrePushTime; + } + public function set serverPrePushTime(value:Number):void + { + _serverPrePushTime = value; + } + private var _serverPreAdapterTime:Number; + /** * Millisecond timestamp of when the server called into the adapter associated with the * destination of this message @@ -145,8 +192,17 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var serverPreAdapterTime:Number; + public function get serverPreAdapterTime():Number + { + return _serverPreAdapterTime; + } + public function set serverPreAdapterTime(value:Number):void + { + _serverPreAdapterTime = value; + } + private var _serverPostAdapterTime:Number; + /** * Millisecond timestamp of when server processing returned from the adapater associated * with the destination of this message @@ -157,8 +213,17 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var serverPostAdapterTime:Number; + public function get serverPostAdapterTime():Number + { + return _serverPostAdapterTime; + } + public function set serverPostAdapterTime(value:Number):void + { + _serverPostAdapterTime = value; + } + private var _serverPreAdapterExternalTime:Number; + /** * Millisecond timestamp of when the adapter associated with the destination of this message * made a call to an external component (for example a JMS server) @@ -169,8 +234,17 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var serverPreAdapterExternalTime:Number; - + public function get serverPreAdapterExternalTime():Number + { + return _serverPreAdapterExternalTime; + } + public function set serverPreAdapterExternalTime(value:Number):void + { + _serverPreAdapterExternalTime = value; + } + + private var _serverPostAdapterExternalTime:Number; + /** * Millisecond timestamp of when processing came back to the adapter associated with the destination * of this message from a call to an external component (for example a JMS server) @@ -181,8 +255,17 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var serverPostAdapterExternalTime:Number; + public function get serverPostAdapterExternalTime():Number + { + return _serverPostAdapterExternalTime; + } + public function set serverPostAdapterExternalTime(value:Number):void + { + _serverPostAdapterExternalTime = value; + } + private var _recordMessageTimes:Boolean; + /** * Flag is true when record-message-times is enabled for the communication channel * @@ -192,7 +275,16 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var recordMessageTimes:Boolean; + public function get recordMessageTimes():Boolean + { + return _recordMessageTimes; + } + public function set recordMessageTimes(value:Boolean):void + { + _recordMessageTimes = value; + } + + private var _recordMessageSizes:Boolean; /** * Flag is true when record-message-sizes is enabled for the communication channel @@ -203,7 +295,14 @@ package mx.messaging.messages * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var recordMessageSizes:Boolean; + public function get recordMessageSizes():Boolean + { + return _recordMessageSizes; + } + public function set recordMessageSizes(value:Boolean):void + { + _recordMessageSizes = value; + } //-------------------------------------------------------------------------- // diff --git a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/RemotingMessage.as b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/RemotingMessage.as index 8ec0216..1eb75f4 100644 --- a/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/RemotingMessage.as +++ b/frameworks/projects/MXRoyale/src/main/royale/mx/messaging/messages/RemotingMessage.as @@ -64,6 +64,8 @@ public class RemotingMessage extends AbstractMessage // //-------------------------------------------------------------------------- + private var _operation:String; + /** * Provides access to the name of the remote method/operation that * should be called. @@ -74,8 +76,17 @@ public class RemotingMessage extends AbstractMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var operation:String; + public function get operation():String + { + return _operation; + } + public function set operation(value:String):void + { + _operation = value; + } + private var _source:String; + /** * This property is provided for backwards compatibility. The best * practice, however, is to not expose the underlying source of a @@ -89,7 +100,14 @@ public class RemotingMessage extends AbstractMessage * @productversion BlazeDS 4 * @productversion LCDS 3 */ - public var source:String; + public function get source():String + { + return _source; + } + public function set source(value:String):void + { + _source = value; + } } }