[MAGNOLIA-1217] RepositorySearchListModel.getSearchPath() not supported by QueryBuilder Created: 16/Nov/06 Updated: 18/Jul/14 Resolved: 12/May/14 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | core |
| Affects Version/s: | 3.0 RC4 |
| Fix Version/s: | None |
| Type: | Bug | Priority: | Major |
| Reporter: | Ali Hammadi | Assignee: | Unassigned |
| Resolution: | Outdated | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
| Date of First Response: |
| Description |
|
While using the method addSelect, it doesn't support the searchPath when it's defined. ERROR info.magnolia.cms.gui.controlx.list.AbstractListModel AbstractListModel.java(getListModelIterator:80) 16.11.2006 18:16:58 can't create the list model iterator, will return an empty list at org.apache.jackrabbit.core.query.sql.JCRSQLQueryBuilder.createQuery(JCRSQLQueryBuilder.java:141) |
| Comments |
| Comment by Ali Hammadi [ 17/Nov/06 ] |
|
Maybe the explanation wasn't clear. I will try to give more details. In all searchlist function (for Website and DMS), I never found any using of the parameter "searchPath" (present in the class RespositorySearchListModel). So I tried to test it in my searchlistModel (which extends RespositorySearchListModel), it generates an InvalidQueryExpression when the query was executed because the expression which has been generated when searchPath was defined is incorrect. It misses a condition (OR, AND, ...) before the jcr:path instruction. So I think the method (addSelect in QueryBuilder class) which generate has to be patched to fix this problem. |
| Comment by Ali Hammadi [ 17/Nov/06 ] |
|
Here is the patch to execute SearchQuery on a defined path |
| Comment by Philipp Bracher [ 17/Nov/06 ] |
|
I will solve this by fixing the advanced search in the enterprise edition |
| Comment by Ali Hammadi [ 17/Nov/06 ] |
|
Here is the patch updated. I forgot to withdraw an instruction not useful. |