[jira] [Issue Comment Edited] (SOLR-2168) Velocity facet output for facet missing

2011-05-17 Thread Erik Hatcher (JIRA)

[ 
https://issues.apache.org/jira/browse/SOLR-2168?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13035015#comment-13035015
 ] 

Erik Hatcher edited comment on SOLR-2168 at 5/17/11 8:05 PM:
-

Alas not yet, Peter.  Sorry.

  was (Author: ehatcher):
Alas not, Peter.  Sorry.
  
 Velocity facet output for facet missing
 ---

 Key: SOLR-2168
 URL: https://issues.apache.org/jira/browse/SOLR-2168
 Project: Solr
  Issue Type: Bug
  Components: Response Writers
Affects Versions: 3.1
Reporter: Peter Wolanin
Priority: Minor
 Attachments: SOLR-2168.patch


 If I add fact.missing to the facet params for a field, the Veolcity output 
 has in the facet list:
 $facet.name (9220)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org



[jira] Issue Comment Edited: (SOLR-2168) Velocity facet output for facet missing

2010-10-17 Thread Peter Wolanin (JIRA)

[ 
https://issues.apache.org/jira/browse/SOLR-2168?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12921852#action_12921852
 ] 

Peter Wolanin edited comment on SOLR-2168 at 10/17/10 10:25 AM:


Those all sound like good changes.

In terms of templating -I'd find something like erb, or PHP, or jsp much 
easier, and I imagine many more people are failiar with those.  So far, I feel 
like it's hard to understand in velocity how variables and control structures 
are distinguished from the output, and it's not clear that it's a real template 
in terms of the way e.g. white space is handled or not.  This is especially 
true in the case of macro output, where is seems like e.g. the carriage returns 
and spaces I'd naturally include in control structures to make them readable 
become part of the output.

The variable handling is also weird, that I need to use #set() for actual 
assignment?

In terms of readablilty, loo for example, at this bit:
{code}
lia href=#url_for_home#lensfq=$esc.url(
{code}

the fq= is output in the middle of a series of macro and function calls but 
nothing visually distinguishes them.  Can I define new functions instead of 
macros?  If a macro call could be written as #{url_for_home} it would provide 
more visual separation. 

I notice in the patch you have:
{code}
-${field.name}:[* TO *]
{code}

Looks like the function call can be written like this?
{code}
${esc.url(-${field.name}:[* TO *])}
{code}


  was (Author: pwolanin):

Those all sound like good changes.

In terms of templating -I'd find something like erb, or PHP, or jsp much 
easier, and I imagine many more people are failiar with those.  So far, I feel 
like it's hard to understand in velocity how variables and control structures 
are distinguished from the output, and it's not clear that it's a real template 
in terms of the way e.g. white space is handled or not.  This is especially try 
in the case of macro output, where is seems like e.g. the carriae returns and 
spaces I'd naturally include in control structures to make them readable become 
part of the output.

The variable handling is also weird, that I need to use #set() for actual 
assignment?

In terms of readablilty, loo for example, at this bit:
{code}
lia href=#url_for_home#lensfq=$esc.url(
{code}

the fq= is output in the middle of a series of macro and function calls but 
nothing visually distinguishes them.  Can I define new functions instead of 
macros?  If a macro call could be written as #{url_for_home} it would provide 
more visual separation. 

I notice in the patch you have:
{code}
-${field.name}:[* TO *]
{code}

Looks like the function call can be written like this?
{code}
${esc.url(-${field.name}:[* TO *])}
{code}

  
 Velocity facet output for facet missing
 ---

 Key: SOLR-2168
 URL: https://issues.apache.org/jira/browse/SOLR-2168
 Project: Solr
  Issue Type: Bug
  Components: Response Writers
Affects Versions: 3.1
Reporter: Peter Wolanin
Priority: Minor
 Attachments: SOLR-2168.patch


 If I add fact.missing to the facet params for a field, the Veolcity output 
 has in the facet list:
 $facet.name (9220)

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


-
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org