Allow empty strings to be filtered.

This commit is contained in:
Nicolas Stepien 2011-10-08 20:23:32 +02:00 committed by James Campos
parent c025cc4007
commit f7c0f5a1e1
2 changed files with 11 additions and 17 deletions

View File

@ -582,9 +582,8 @@
}, },
name: function(root) { name: function(root) {
var name; var name;
if ((name = root.className === 'op' ? $('.postername', root).textContent : $('.commentpostername', root).textContent)) { name = root.className === 'op' ? $('.postername', root) : $('.commentpostername', root);
return filter.test('name', name); return filter.test('name', name.textContent);
}
}, },
trip: function(root) { trip: function(root) {
var trip; var trip;
@ -600,17 +599,13 @@
}, },
sub: function(root) { sub: function(root) {
var sub; var sub;
if ((sub = root.className === 'op' ? $('.filetitle', root).textContent : $('.replytitle', root).textContent)) { sub = root.className === 'op' ? $('.filetitle', root) : $('.replytitle', root);
return filter.test('sub', sub); return filter.test('sub', sub.textContent);
}
}, },
com: function(root) { com: function(root) {
var com; return filter.test('com', ($.el('a', {
if (com = ($.el('a', {
innerHTML: $('blockquote', root).innerHTML.replace(/<br>/g, '\n') innerHTML: $('blockquote', root).innerHTML.replace(/<br>/g, '\n')
})).textContent) { })).textContent);
return filter.test('com', com);
}
}, },
file: function(root) { file: function(root) {
var file; var file;

View File

@ -415,8 +415,8 @@ filter =
return true if regexp.test value return true if regexp.test value
name: (root) -> name: (root) ->
if (name = if root.className is 'op' then $('.postername', root).textContent else $('.commentpostername', root).textContent) name = if root.className is 'op' then $ '.postername', root else $ '.commentpostername', root
filter.test 'name', name filter.test 'name', name.textContent
trip: (root) -> trip: (root) ->
if trip = $ '.postertrip', root if trip = $ '.postertrip', root
filter.test 'trip', trip.textContent filter.test 'trip', trip.textContent
@ -424,11 +424,10 @@ filter =
if mail = $ '.linkmail', root if mail = $ '.linkmail', root
filter.test 'mail', mail.href filter.test 'mail', mail.href
sub: (root) -> sub: (root) ->
if (sub = if root.className is 'op' then $('.filetitle', root).textContent else $('.replytitle', root).textContent) sub = if root.className is 'op' then $ '.filetitle', root else $ '.replytitle', root
filter.test 'sub', sub filter.test 'sub', sub.textContent
com: (root) -> com: (root) ->
if com = ($.el 'a', innerHTML: $('blockquote', root).innerHTML.replace /<br>/g, '\n').textContent filter.test 'com', ($.el 'a', innerHTML: $('blockquote', root).innerHTML.replace /<br>/g, '\n').textContent
filter.test 'com', com
file: (root) -> file: (root) ->
if file = $ '.filesize span', root if file = $ '.filesize span', root
filter.test 'file', file.title filter.test 'file', file.title