Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.0
    • Fix Version/s: None
    • Component/s: core/search
    • Labels:
      None

      Description

      If you want an HTML representation of an Explanation, you might call the toHtml() method. However, the output of this method looks like the following:

      <ul>
      <li>some value = some description</li>
      <ul>
      <li>some nested value = some description</li>
      </ul>
      </ul>

      As it is illegal in HTML to nest a UL directly inside a UL, this method will always output unparseable HTML if there are nested explanations.

      What Lucene probably means to output is the following, which is valid HTML:

      <ul>
      <li>some value = some description
      <ul>
      <li>some nested value = some description</li>
      </ul>
      </li>
      </ul>

        Activity

        Hide
        lucenebugs@danielnaber.de Daniel Naber added a comment -

        Sorry, I must have looked at the wrong output. You're right, it seems to be okay now.

        Show
        lucenebugs@danielnaber.de Daniel Naber added a comment - Sorry, I must have looked at the wrong output. You're right, it seems to be okay now.
        Hide
        lucenebugs@danielnaber.de Daniel Naber added a comment -

        Sorry, I think the fix in not correct. Output now starts with "<li>", but "<li>" must always be inside "<ul>".

        Show
        lucenebugs@danielnaber.de Daniel Naber added a comment - Sorry, I think the fix in not correct. Output now starts with "<li>", but "<li>" must always be inside "<ul>".
        Hide
        hossman Hoss Man added a comment -

        thanks forspotting this. I've commited a fix that basically does what you suggest, but includes a <br /> between the description and hte nested explanations.

        Show
        hossman Hoss Man added a comment - thanks forspotting this. I've commited a fix that basically does what you suggest, but includes a <br /> between the description and hte nested explanations.

          People

          • Assignee:
            hossman Hoss Man
            Reporter:
            trejkaz Trejkaz
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development