[jira] [Commented] (OFBIZ-6595) Have the ability to test solr queries through widgets (screens & forms)

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

[jira] [Commented] (OFBIZ-6595) Have the ability to test solr queries through widgets (screens & forms)

Nicolas Malin (Jira)

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

Michael Brohl commented on OFBIZ-6595:
--------------------------------------

I just stumbled upon this during my work on OFBIZ-9440.

As Amardeep said, the functionality is incomplete. For the average user, the query widget pretends to provide a search functionality but there is nothing more than this widget (like Pierre rightly mentioned providing the patch).
I see no reason why this work was committed in this state because it only causes confusion. We should definitely be more cautious when adding new functionality to the codebase.

In a case like this, we should have waited until the work provides a minimum of functionality and ist tested accordingly.

I will add this as a subtask to the refactoring issue and work out a minimal functional solution in the course of this work.

> Have the ability to test solr queries through widgets (screens & forms)
> -----------------------------------------------------------------------
>
>                 Key: OFBIZ-6595
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-6595
>             Project: OFBiz
>          Issue Type: Improvement
>          Components: solr
>    Affects Versions: Trunk
>            Reporter: Pierre Smits
>            Assignee: Jacques Le Roux
>         Attachments: OFBIZ-6595-r1.patch
>
>
> Adding widgest (screens, forms, etc) so that an end-user can execute a search  against the data indexed would enhance the applicability of the application.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)