I have spotted something strange when trying to connect multiple conditions with OR op

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

Unexpected behavior when using multiple conditions about defiant.js HOT 3 CLOSED

BlackWiCKED commented on August 17, 2024

Unexpected behavior when using multiple conditions

from defiant.js.

Comments (3)

hbi99 commented on August 17, 2024

Hi @BlackWiCKED,
Thanks for the issue...actually, there is an regexp that automatically changes contains to case-insensitive. Look at this file;

https://github.com/hbi99/defiant.js/blob/master/src/string.js

I can fix this in a couple of hours...

from defiant.js.

BlackWiCKED commented on August 17, 2024

Actually, looks like it was easier to find than I thought. There is only one missing /g in the replace function to replace all occurrences of contains(). It works now.

https://raw.githubusercontent.com/hbi99/defiant.js/master/dist/defiant-latest.js

/contains\(([^,]+),([^\\)]+)\)/g

if (!String.prototype.xTransform) {
    String.prototype.xTransform = function () {
        var str = this;
        if (this.indexOf('translate(') === -1) {
            str = this.replace(/contains\(([^,]+),([^\\)]+)\)/g, function(c,h,n) {
                var a = 'abcdefghijklmnopqrstuvwxyz',
                    q = n.trim().slice(-1);
                return "contains(translate("+ h +", "+ q + a.toUpperCase() + q +", "+ q + a + q +"),"+ n.toLowerCase() +")";
            });
        }
        return str.toString();
    };
}

from defiant.js.

BlackWiCKED commented on August 17, 2024

Shall I send a proper pull request? I have not forked the original repository yet.

from defiant.js.

Recommend Projects