Skip to content
This repository has been archived by the owner on May 16, 2018. It is now read-only.

Fixed #92 #558

Closed
wants to merge 1 commit into from
Closed

Fixed #92 #558

wants to merge 1 commit into from

Conversation

bevalorous
Copy link

@froschdesign
Copy link
Member

Your solution doesn't work.

Here is an unit test:

/**
 * @group GH-92
 */
public function testHtmlHighlightingWithGermanUmlaut()
{
    $doc = Zend_Search_Lucene_Document_Html::loadHTML(
        '<HTML><BODY>Überfall</BODY></HTML>'
    );
    $doc->highlight('Überfall');
    $this->assertContains(
        '<b style="color:black;background-color:#66ffff">&Uuml;berfall</b>',
        $doc->getHTML()
    );
}

Result:

Failed asserting that
'<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">
<html><body>&Uuml;<b style="color:black;background-color:#66ffff">berfall</b></body></html>'
contains
"<b style="color:black;background-color:#66ffff">&Uuml;berfall</b>".

@froschdesign
Copy link
Member

ping @bevalorous

@froschdesign
Copy link
Member

Closing due to lack of response.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
awaiting author updates no signed CLA
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bevalorous @froschdesign