Skip to content
This repository has been archived by the owner on Oct 2, 2019. It is now read-only.

Cast other data types to string when highlighting #1027

Closed
wants to merge 1 commit into from
Closed

Cast other data types to string when highlighting #1027

wants to merge 1 commit into from

Conversation

slawekkolodziej
Copy link

This PR fixes 'undefined is not a function' error, when you try to pass number through "highlight" filter, like here:

<div ng-bind-html="person.age | highlight: $select.search"></div> 

I looked for possibly related issues and I found these: #517 and #555

@cdjackson
Copy link
Contributor

Possible duplicate of #1008

@slawekkolodziej
Copy link
Author

Yup, it appears to be a duplicate. The only difference is that mine PR provides unit tests. Anyway, why don't you merge any of them to actually close the issue?

@cdjackson
Copy link
Contributor

Anyway, why don't you merge any of them to actually close the issue?

Because when I wrote this I was just trying to help the maintainers out by identifying duplicates with a view to tidying up the repository...
Now I have access to the repository with a view to refactoring it (see #1096 / #1108) and I'll add this there (#1134).

@slawekkolodziej
Copy link
Author

Ahh I see! I assumed that this project was dead, but it definitely needed guys like you 👍

@user378230
Copy link
Contributor

#1402 merged so closing this.

@user378230 user378230 closed this Mar 20, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants