Bug 703585 - Update Search box to support using comma as a phrase separator in addition to space
Summary: Update Search box to support using comma as a phrase separator in addition to...
Keywords:
Status: NEW
Alias: None
Product: RHQ Project
Classification: Other
Component: Core UI
Version: 4.0.0
Hardware: Unspecified
OS: Unspecified
low
unspecified
Target Milestone: ---
: ---
Assignee: Nobody
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: jon3
TreeView+ depends on / blocked
 
Reported: 2011-05-10 18:09 UTC by Charles Crouch
Modified: 2024-03-04 13:34 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Clone Of:
Environment:
Last Closed:
Embargoed:


Attachments (Terms of Use)

Description Charles Crouch 2011-05-10 18:09:37 UTC
This is running against RHQ4.0.0 RC

1) Inventory the RHQ server and all its child services
2) Go to the Inventory link at the top
3) Click the Services link in the top left hand pane. The right hand side refreshes with a list of services. The Search box is prefilled with "category=service", without the quotes.
4) In the Search box Start appending to the end of "category=service"  the following ", name=TestBean". As you type the number of items in the autosuggest dropdown should change to finally show 7 items.
5) Hit the Enter key

Expectation is that the the 7 items are displayed in the list.
But instead you get the exception below logged in the message center.
Removing the "category=service," part of the search and hitting Enter returns the correct results.

java.lang.RuntimeException:[1305050827193] javax.ejb.EJBException:org.rhq.enterprise.server.search.SearchExpressionException: No enum of type 'ResourceCategory' with name matching 'service,' -> org.rhq.enterprise.server.search.SearchExpressionException:No enum of type 'ResourceCategory' with name matching 'service,'
--- STACK TRACE FOLLOWS ---
[1305050827193] javax.ejb.EJBException:org.rhq.enterprise.server.search.SearchExpressionException: No enum of type 'ResourceCategory' with name matching 'service,' -> org.rhq.enterprise.server.search.SearchExpressionException:No enum of type 'ResourceCategory' with name matching 'service,'
    at Unknown.java_lang_RuntimeException_$RuntimeException__Ljava_lang_RuntimeException_2Ljava_lang_RuntimeException_2(Unknown source:0)
    at Unknown.com_google_gwt_user_client_rpc_core_java_lang_RuntimeException_1FieldSerializer_instantiate__Lcom_google_gwt_user_client_rpc_SerializationStreamReader_2Ljava_lang_RuntimeException_2(Unknown source:0)
    at Unknown.com_google_gwt_user_client_rpc_impl_SerializerBase$MethodMap_$instantiate__Lcom_google_gwt_user_client_rpc_impl_SerializerBase$MethodMap_2Lcom_google_gwt_user_client_rpc_SerializationStreamReader_2Ljava_lang_String_2Ljava_lang_Object_2(Unknown source:0)
    at Unknown.com_google_gwt_user_client_rpc_impl_SerializerBase_$instantiate__Lcom_google_gwt_user_client_rpc_impl_SerializerBase_2Lcom_google_gwt_user_client_rpc_SerializationStreamReader_2Ljava_lang_String_2Ljava_lang_Object_2(Unknown source:0)
    at Unknown.com_google_gwt_user_client_rpc_impl_AbstractSerializationStreamReader_$readObject__Lcom_google_gwt_user_client_rpc_impl_AbstractSerializationStreamReader_2Ljava_lang_Object_2(Unknown source:0)
    at Unknown.com_google_gwt_user_client_rpc_impl_RequestCallbackAdapter_$onResponseReceived__Lcom_google_gwt_user_client_rpc_impl_RequestCallbackAdapter_2Lcom_google_gwt_http_client_Request_2Lcom_google_gwt_http_client_Response_2V(Unknown source:0)
    at Unknown.org_rhq_enterprise_gui_coregui_client_util_rpc_TrackingRequestCallback_onResponseReceived__Lcom_google_gwt_http_client_Request_2Lcom_google_gwt_http_client_Response_2V(Unknown source:0)
    at Unknown.com_google_gwt_http_client_Request_$fireOnResponseReceived__Lcom_google_gwt_http_client_Request_2Lcom_google_gwt_http_client_RequestCallback_2V(Unknown source:0)
    at Unknown.com_google_gwt_http_client_RequestBuilder$1_onReadyStateChange__Lcom_google_gwt_xhr_client_XMLHttpRequest_2V(Unknown source:0)
    at Unknown.anonymous(Unknown source:0)
    at Unknown.com_google_gwt_core_client_impl_Impl_entry0__Ljava_lang_Object_2Ljava_lang_Object_2Ljava_lang_Object_2Ljava_lang_Object_2(Unknown source:0)
    at Unknown.anonymous(Unknown source:0)
    at Unknown.anonymous(Unknown source:0)

Comment 1 Mike Foley 2011-05-10 18:19:22 UTC
repro'd issue in rhq 4 master branch build from 5/9

Comment 2 Charles Crouch 2011-05-10 18:32:33 UTC
(1:21:57 PM) mazz: ccrouch: what if you did "category=server"
(1:21:59 PM) mazz: does that work?
(1:25:42 PM) ccrouch: mazz: nope, blows up similarly
(1:25:43 PM) ccrouch: java.lang.RuntimeException:[1305051914463]  javax.ejb.EJBException:org.rhq.enterprise.server.search.SearchExpressionException:  No enum of type 'ResourceCategory' with name matching 'server,' ->  org.rhq.enterprise.server.search.SearchExpressionException:No enum of  type 'ResourceCategory' with name matching 'server,'
(1:25:51 PM) mazz: how can that be?
(1:26:05 PM) mazz: if you go to Inventory>Servers
(1:26:16 PM) mazz: what does the search bar say inside it? (I don't have a server running at the moment)
(1:26:17 PM) ccrouch: "category=server, name=TestBean" blows up
(1:26:17 PM) ccrouch: "category=server" works
(1:26:25 PM) mazz: oh
(1:26:32 PM) mazz: don't use comma :)
(1:26:40 PM) mazz: separate clauses with space
(1:26:45 PM) mazz: it things you want "server,"
(1:26:51 PM) mazz: literally, with a comma after server
(1:27:00 PM) mazz: so yes, there is no category of type "server,"
(1:27:06 PM) ccrouch: ha
(1:27:12 PM) ccrouch: that was it
(1:27:23 PM) ccrouch: shows how long it is since i used the search bar
(1:27:32 PM) mazz: I still make that mistake
(1:27:38 PM) ccrouch:  but thats not super intuitive though
(1:27:42 PM) mazz: "does it want space? or comma? or semi?"
(1:27:46 PM) ccrouch: that whitespace is the only separator
(1:28:01 PM) mazz: I agree. like I said, I get confused too. now I just always assume space
(1:28:10 PM) ccrouch: ok, i'm turning this into an RFE
(1:28:25 PM) lkrejci: mazz: spinder: the commit hashes stay the same after the push..
(1:28:27 PM) mazz: I can tell you right now that's probably gonna be a pita to attempt to fix
(1:28:28 PM) ccrouch: i think we should support whitespace and comma atleast
(1:29:08 PM) ccrouch: mazz: haven't i told you about "solution tuesdays", you cant tell me about any problems on Tuesdays only solutions
(1:29:53 PM) mazz: the solution is to inform the user of PEBAK and to use proper syntax.
(1:29:55 PM) mazz: :p
(1:31:18 PM) ccrouch: ha


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