Project

General

Profile

Bug #3407

Mulighed for at sætte facets ved søgning

Added by Jesper Kristensen over 2 years ago. Updated over 2 years ago.

Status:
Resolved (tag version)
Priority:
High
Estimated time:
URL med eksempel:
Kategorier:
Søgeabstraktionslag (SAL)

Description

Jeg ser ikke muligheder for at sætte andre facets gennem SAL end dem den henter fra ding_facetbrowser.

 

Facets array findes på TingSearchRequest https://github.com/ding2/ding2/blob/master/modules/ting/src/Search/TingSearchRequest.php#L52 men bruges ikke i https://github.com/ding2/ding2/blob/master/modules/opensearch/includes/opensearch.search.inc#L116 som kalder opensearch_do_search som har options array som hvis ikke indholder facets autoamtisk loader facets fra ding_facetbrowser_facets variable.

Jeg har brug for denne mulighed, så PR er på vej.

History

#1 Updated by Jesper Kristensen over 2 years ago

Dette er krævet for at kan benytte kampage plus projektet ellers kan man ikke benytte facet match regler, som er hoved features ved det nye campaign modul.

#2 Updated by Gitte Barlach over 2 years ago

  • Status changed from New to Development

#3 Updated by Jesper Kristensen over 2 years ago

  • Status changed from Development to Needs code review
  • Assignee changed from Jesper Kristensen to Kasper Garnæs

#4 Updated by Gitte Barlach over 2 years ago

  • Priority changed from Normal to High

#5 Updated by Kasper Garnæs over 2 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Jesper Kristensen

Reviewed. Jeg har et par kommentarer og spørgsmål.

#6 Updated by Jesper Kristensen over 2 years ago

For god ordens skyld vil jeg lige sige at vi/jeg bruger dette til at bestemme hvilke "facets most comman values" der skal tigger en campaign.

Alle facets er (efter ønske fra kunden) de-koblet fra ding_facetbrowser og derfor har jeg brug for at definere hvilke der skal bruges (dette er selvfølgelig konfigureret i ding_campaign_facets modulet) ved en given søgning i campaign triggers.

Se https://github.com/aakb/ding2/blob/feature/2177-kampange-plus/modules/ding_campaign_plus/modules/ding_campaign_plus_facet/ding_campaign_plus_facet.module#L926 

#7 Updated by Jesper Kristensen over 2 years ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Jesper Kristensen to Kasper Garnæs

Lavet til rettelse ifh. til review'et og er nu klar til re-review.

#8 Updated by Kasper Garnæs over 2 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to Jesper Kristensen

Jeg har stadig et par kommentarer til dokumentationen.

#9 Updated by Jesper Kristensen over 2 years ago

  • Status changed from Reviewed - Needs info/rework to Needs code review
  • Assignee changed from Jesper Kristensen to Kasper Garnæs

Har givet den en omgang mere i maskinen.

#10 Updated by Kasper Garnæs over 2 years ago

  • Status changed from Needs code review to Technical test
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Godkendt og merged.

#11 Updated by Michael W. Christoffersen over 2 years ago

  • Assignee changed from Gitte Barlach to Jesper Nørskov Søndergaard

@Jesper - hvordan testes dette?

#12 Updated by Michael W. Christoffersen over 2 years ago

  • Assignee changed from Jesper Nørskov Søndergaard to Jesper Kristensen

#13 Updated by Gitte Barlach over 2 years ago

  • Status changed from Technical test to Resolved (tag version)

Der er tale om en teknisk ændring der kun anvendes ift. det kommende kampagne+ modul. Så godkendelsen at dette issue må her og nu bero på den tekniske godkendelse der er foretaget som følge af code review og merge til core.

Also available in: Atom PDF