Bug 1378723 - [GSS](6.4.z) generic-jms-ra's JmsMCFProperties.getSessionDefaultType returns incorrect value for session type
Summary: [GSS](6.4.z) generic-jms-ra's JmsMCFProperties.getSessionDefaultType returns ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Enterprise Application Platform 6
Classification: JBoss
Component: JMS
Version: 6.4.10
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: CR1
: EAP 6.4.12
Assignee: Tom Ross
QA Contact: Peter Mackay
URL:
Whiteboard:
Depends On:
Blocks: 1379572 eap6412-payload 1379571 1380885
TreeView+ depends on / blocked
 
Reported: 2016-09-23 08:16 UTC by varsha
Modified: 2020-07-16 08:54 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1379571 1379572 (view as bug list)
Environment:
Last Closed: 2017-01-17 13:10:00 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker JBEAP-6266 0 Major Verified [GSS](7.1.0) generic-jms-ra's JmsMCFProperties.getSessionDefaultType returns incorrect value for session type 2017-06-22 14:19:47 UTC
Red Hat Issue Tracker JBEAP-6267 0 Major Verified [GSS](7.0.z) generic-jms-ra's JmsMCFProperties.getSessionDefaultType returns incorrect value for session type 2017-06-22 14:19:47 UTC
Red Hat Issue Tracker WFLY-7202 0 Major Resolved generic-jms-ra's JmsMCFProperties.getSessionDefaultType returns incorrect value for session type 2017-06-22 14:19:47 UTC
Red Hat Knowledge Base (Article) 2651201 0 None None None 2016-09-23 09:08:14 UTC

Description varsha 2016-09-23 08:16:44 UTC
Description of problem: generic-jms-ra's JmsMCFProperties.getSessionDefaultType returns incorrect value for session type 

Actual code:

  public void setSessionDefaultType(String type)
    throws ResourceException
  {
    if (type.equals(QUEUE_TYPE))
      this.type = 1;
    else if (type.equals(TOPIC_TYPE))
      this.type = 2;
    else
      this.type = 0;
  }

  public String getSessionDefaultType() {
    if (this.type == 0)
      return "agnostic";
    if (this.type == 1) {
      return TOPIC_TYPE;
    }
    return QUEUE_TYPE;
  }




Expected code:

  public void setSessionDefaultType(String type)
    throws ResourceException
  {
    if (type.equals(QUEUE_TYPE))
      this.type = 1;
    else if (type.equals(TOPIC_TYPE))
      this.type = 2;
    else
      this.type = 0;
  }

  public String getSessionDefaultType() {
    if (this.type == 0)
      return "agnostic";
    if (this.type == 1) {
      return QUEUE_TYPE;
    }
    return TOPIC_TYPE;
  }

Comment 4 Peter Mackay 2016-11-22 16:14:32 UTC
Verified with EAP 6.4.12.CP.CR1

Comment 5 Petr Penicka 2017-01-17 13:10:00 UTC
Retroactively bulk-closing issues from released EAP 6.4 cummulative patches.


Note You need to log in before you can comment on or make changes to this bug.