Changeset 4966


Ignore:
Timestamp:
04/14/14 12:22:02 (10 years ago)
Author:
Twan Goosen
Message:

added and applied web app query parameter constants 'fq','q','docId'

Location:
vlo/branches/vlo-3.0/vlo-web-app/src
Files:
7 edited

Legend:

Unmodified
Added
Removed

  • vlo/branches/vlo-3.0/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/VloWebAppParameters.java

    r4919 r4966  
    2424public class VloWebAppParameters {
    2525    public final static String THEME = "theme";
     26    public final static String DOCUMENT_ID = "docId";
     27    public static final String QUERY = "q";
     28    public static final String FILTER_QUERY = "fq";
    2629}

  • vlo/branches/vlo-3.0/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/service/solr/impl/QueryFacetsSelectionParametersConverter.java

    r4965 r4966  
    2727import org.apache.wicket.request.mapper.parameter.PageParameters;
    2828import org.apache.wicket.util.string.StringValue;
     29import static eu.clarin.cmdi.vlo.VloWebAppParameters.*;
    2930
    3031/**
    … …  
    3940        // Assuming AND. TODO: decode NOT,OR,not empty. Abandon multimap stategy?
    4041        // Get query string from params
    41         final String query = params.get("q").toOptionalString();
     42        final String query = params.get(QUERY).toOptionalString();
    4243
    4344        // Get facet selections from params
    44         final List<StringValue> facetValues = params.getValues("fq");
     45        final List<StringValue> facetValues = params.getValues(FILTER_QUERY);
    4546
    4647        final HashMap<String, FacetSelection> selection = Maps.newHashMapWithExpectedSize(facetValues.size());
    … …  
    7172        final String query = selection.getQuery();
    7273        if (query != null) {
    73             params.add("q", query);
     74            params.add(QUERY, query);
    7475        }
    7576
    … …  
    7980            //TODO: encode NOT,OR
    8081            for (String value : facetSelection.getValue().getValues()) {
    81                 params.add("fq", String.format("%s:%s", facetSelection.getKey(), value));
     82                params.add(FILTER_QUERY, String.format("%s:%s", facetSelection.getKey(), value));
    8283            }
    8384        }

  • vlo/branches/vlo-3.0/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/wicket/pages/RecordPage.java

    r4913 r4966  
    1818
    1919import eu.clarin.cmdi.vlo.FacetConstants;
     20import eu.clarin.cmdi.vlo.VloWebAppParameters;
    2021import eu.clarin.cmdi.vlo.pojo.QueryFacetsSelection;
    2122import eu.clarin.cmdi.vlo.pojo.SearchContext;
    … …  
    8485        selectionModel = Model.of(selection);
    8586
    86         final StringValue docId = params.get("docId");
     87        final StringValue docId = params.get(VloWebAppParameters.DOCUMENT_ID);
    8788        if (docId.isEmpty()) {
    8889            Session.get().error("No document ID provided");

  • vlo/branches/vlo-3.0/vlo-web-app/src/main/java/eu/clarin/cmdi/vlo/wicket/panels/TopLinksPanel.java

    r4930 r4966  
    168168
    169169            if (documentModel != null) {
    170                 params.add("docId", documentModel.getObject().getFirstValue(FacetConstants.FIELD_ID));
     170                params.add(VloWebAppParameters.DOCUMENT_ID, documentModel.getObject().getFirstValue(FacetConstants.FIELD_ID));
    171171            }
    172172

  • vlo/branches/vlo-3.0/vlo-web-app/src/test/java/eu/clarin/cmdi/vlo/service/solr/impl/SolrDocumentQueryFactoryImplTest.java

    r4963 r4966  
    7575    @Test
    7676    public void testCreateDocumentQueryForDocId() {
    77         final SolrQuery query = instance.createDocumentQuery("docId");
     77        final SolrQuery query = instance.createDocumentQuery("documentId");
    7878
    7979        final String[] filterQueries = query.getFilterQueries();
    8080        assertEquals(1, filterQueries.length);
    81         assertEquals("id:\"docId\"", filterQueries[0]);
     81        assertEquals("id:\"documentId\"", filterQueries[0]);
    8282
    8383        final String fields = query.getFields();

  • vlo/branches/vlo-3.0/vlo-web-app/src/test/java/eu/clarin/cmdi/vlo/service/solr/impl/SolrDocumentServiceImplTest.java

    r4963 r4966  
    6161        final SolrQuery solrQuery = new SolrQuery("query");
    6262        final SolrDocument resultDocument = new SolrDocument();
    63         resultDocument.setField("id", "docId");
     63        resultDocument.setField("id", "documentId");
    6464        final SolrDocumentList resultList = new SolrDocumentList();
    6565        resultList.add(resultDocument);
    6666        context.checking(new Expectations() {
    6767            {
    68                 oneOf(queryFactory).createDocumentQuery("docId");
     68                oneOf(queryFactory).createDocumentQuery("documentId");
    6969                will(returnValue(solrQuery));
    7070                oneOf(dao).getDocuments(solrQuery);
    … …  
    7373        });
    7474
    75         final SolrDocument result = instance.getDocument("docId");
     75        final SolrDocument result = instance.getDocument("documentId");
    7676        assertEquals(resultDocument, result);
    77         assertEquals("docId", result.getFieldValue("id"));
     77        assertEquals("documentId", result.getFieldValue("id"));
    7878    }
    7979

  • vlo/branches/vlo-3.0/vlo-web-app/src/test/java/eu/clarin/cmdi/vlo/wicket/pages/TestRecordPage.java

    r4661 r4966  
    22
    33import eu.clarin.cmdi.vlo.FacetConstants;
     4import eu.clarin.cmdi.vlo.VloWebAppParameters;
    45import eu.clarin.cmdi.vlo.VloWicketApplication;
    56import eu.clarin.cmdi.vlo.config.DefaultVloConfigFactory;
    … …  
    7273        document = new SolrDocument();
    7374        params = new PageParameters();
    74         params.set("docId", "docId");
     75        params.set(VloWebAppParameters.DOCUMENT_ID, "documentId");
    7576    }
    7677
    … …  
    7980        mockery.checking(new Expectations() {
    8081            {
    81                 oneOf(documentService).getDocument("docId");
     82                oneOf(documentService).getDocument("documentId");
    8283                will(returnValue(document));
    8384            }
    … …  
    9293        mockery.checking(new Expectations() {
    9394            {
    94                 oneOf(documentService).getDocument("docId");
     95                oneOf(documentService).getDocument("documentId");
    9596                will(returnValue(document));
    9697            }
    … …  
    107108        mockery.checking(new Expectations() {
    108109            {
    109                 oneOf(documentService).getDocument("docId");
     110                oneOf(documentService).getDocument("documentId");
    110111                will(returnValue(document));
    111112            }
Note: See TracChangeset for help on using the changeset viewer.