Search code examples
phppreg-replacesanitizationpreg-replace-callback

Convert a function from preg_replace to preg_replace_callback()


I need to convert preg_replace() to preg_replace_callback() in this function of an outdated CMS extension:

// santizes a regex pattern
private static function sanitize( $pattern, $m = false, $e = false ) {
    if( preg_match( '/^\/(.*)([^\\\\])\/(.*?)$/', $pattern, $matches ) ) {
        $pat = preg_replace(
            '/([^\\\\])?\(\?(.*\:)?(.*)\)/Ue',
            '\'$1(?\' . self::cleanupInternal(\'$2\') . \'$3)\'',
            $matches[1] . $matches[2]
        );
        $ret = '/' . $pat . '/';
        if( $m ) {
            $mod = '';
            foreach( self::$modifiers as $val ) {
                if( strpos( $matches[3], $val ) !== false ) {
                    $mod .= $val;
                }
            }
            if( !$e ) {
                $mod = str_replace( 'e', '', $mod );
            }
            $ret .= $mod;
        }
    } else {
        $pat = preg_replace(
            '/([^\\\\])?\(\?(.*\:)?(.*)\)/Ue',
            '\'$1(?\' . self::cleanupInternal(\'$2\') . \'$3)\'',
            $pattern
        );
        $pat = preg_replace( '!([^\\\\])/!', '$1\\/', $pat );
        $ret = '/' . $pat . '/';
    }
    return $ret;
}

I can only imagine what this function does. I tried this but it didsn't work:

private static function sanitize( $pattern, $m = false, $e = false ) {
    if( preg_match( '/^\/(.*)([^\\\\])\/(.*?)$/', $pattern, $matches ) ) {
        $pat = preg_replace_callback(
            '/([^\\\\])?\(\?(.*\:)?(.*)\)/U',
            function($matches) {return CallFunction('\'$1(?\' . self::cleanupInternal(\'$2\') . \'$3)\''); },
            $matches[1] . $matches[2]
        );
        $ret = '/' . $pat . '/';
        if( $m ) {
            $mod = '';
            foreach( self::$modifiers as $val ) {
                if( strpos( $matches[3], $val ) !== false ) {
                    $mod .= $val;
                }
            }
            if( !$e ) {
                $mod = str_replace( 'e', '', $mod );
            }
            $ret .= $mod;
        }
    } else {
        $pat = preg_replace_callback(
            '/([^\\\\])?\(\?(.*\:)?(.*)\)/U',
        function($matches) {return CallFunction('\'$1(?\' . self::cleanupInternal(\'$2\') . \'$3)\''); },
            $pattern
        );
        $pat = preg_replace( '!([^\\\\])/!', '$1\\/', $pat );
        $ret = '/' . $pat . '/';
    }
    return $ret;
}

Could somebody help me on this?


Solution

  • Without any certitude, you can try this for the first preg_replace, and modify the second preg_replace in a same way:

    $that = $this;
    $pat = preg_replace_callback(
                '/([^\\\\])?\(\?(.*:)?(.*)\)/U',
                function ($m) use ($that) {
                    return  $m[1] . '(?' . $that->cleanupInternal($m[2]) . $m[3];
                },
                $matches[1] . $matches[2]
    );
    

    As an aside comment, I don't think that ([^\\\\])? has any sense or is useful for something since it is optional and reuse in the replacement string at the same position.