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

Chris M. Hostetter commented on SOLR-17009:
-------------------------------------------

{quote}but i don't know if there is a similar concern / indenter option with 
arrays?
{quote}
Oh, i see ... there is also a {{withArrayIndenter(...)}} method, but 
{{JacksonJsonWriter}} explicitly No-Ops that...
{code:java}
   protected static final DefaultPrettyPrinter pretty =
      new DefaultPrettyPrinter()
          .withoutSpacesInObjectEntries()
          .withArrayIndenter(DefaultPrettyPrinter.NopIndenter.instance);
{code}
 

i don't really understand the choice to no-op array indentation but not leave 
object indentation ... but if that's intentional, and not just an oversight, 
replacing the object indentation with {{new 
DefaultIndenter().withLinefeed("\n")}} is probably the right course of action.

> json.wrf parameter ignored in JacksonJsonWriter
> -----------------------------------------------
>
>                 Key: SOLR-17009
>                 URL: https://issues.apache.org/jira/browse/SOLR-17009
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: JSON Request API
>    Affects Versions: 9.3
>            Reporter: Chris Stewart
>            Assignee: Alex Deparvu
>            Priority: Blocker
>             Fix For: 9.4
>
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> When requesting json output in 9.3 using json.wrf to trigger a JSONP request, 
> the return is not wrapped in a function and is instead returned as a plain 
> JSON object
>  My test cases are:
> wt=json&json.wrf=ng_jsonp_callback_0 on Solr 9.3.0
>  { "responseHeader":{ "status":0, "QTime":9, "params":{
>  wt=json&json.wrf=ng_jsonp_callback_0 on Solr 7.3.1
>  ng_jsonp_callback_0({ "responseHeader":{ "status":400, "QTime":206, 
> "params":{



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to