pass preg replace variable to function

Question

This is my function:

function bbc2html($content) {
    $search = array (
      '/(\[b\])(.*?)(\[\/b\])/',
      '/(\[u\])(.*?)(\[\/u\])/',
      '/(\[i\])(.*?)(\[\/i\])/',
      '/(\[youtube\])(.*?)(\[\/youtube\])/'
    );
  
    $replace = array (
      '<b>$2</b>',
      '<u>$2</u>',
      '<i>$2</i>',
      '<div class="video_wrapper card border-0 shadow text-center m-2" style="width: 25rem;">
       <div class="video_trigger card-body" data-source="'.getYoutubeVideoID('$2').'" data-type="youtube">
          <p class="text-center">As soon as you click on the button you accept that cookies from YouTube to be loaded</p>
          <input type="button" class="btn btn-success" value="OK">
          <a href="$2" type="button" target="_blank" class="btn btn-danger"><i class="fa-brands fa-youtube pe-1"></i>direct link</a>
       </div>
       <div class="video_layer" style="display: none;">
          <iframe src="" border="0" data-scaling="true" data-format="16:9" style="height: 0px;"></iframe>
       </div>
       </div>'
    );
    
    return preg_replace($search, $replace , $content);
  }

function getYoutubeVideoID($link):string{
    $baseUrl = parse_url($link, PHP_URL_HOST); 

    if($baseUrl == "youtu.be"){
        $urlParts = explode('/',$link);
        $id = end($urlParts);
    } elseif (preg_match('/(youtube.com)/i',$baseUrl)) {
        if(preg_match('/(youtube.com\/watch\?v=)/i',$link)){
            $urlParts = explode('?v=',$link);
            $id = end($urlParts);
        } elseif(preg_match('/(youtube.com\/)/i',$link)) {
            $urlParts = explode('/',$link);
            $id = end($urlParts);
        } else {
            return "";
        }
    } else {
        return "";
    }

    return $id;
}

I want to pass the variable ($2, contains the link to the youtube video) to the function (getYoutubeVideoID, returns only the id of the video), but it just pass '$2' instead of the link.

Example: [youtube] https://www.youtube.com/watch?v=LXb3EKWsInQ [/youtube]

This should be replaced with:

<div class="video_wrapper card border-0 shadow text-center m-2" style="width: 25rem;">
       <div class="video_trigger card-body" data-source="LXb3EKWsInQ" data-type="youtube">
          <p class="text-center">As soon as you click on the button you accept that cookies from YouTube to be loaded</p>
          <input type="button" class="btn btn-success" value="OK">
          <a href="https://www.youtube.com/watch?v=LXb3EKWsInQ" type="button" target="_blank" class="btn btn-danger"><i class="fa-brands fa-youtube pe-1"></i>direct link</a>
       </div>
       <div class="video_layer" style="display: none;">
          <iframe src="" border="0" data-scaling="true" data-format="16:9" style="height: 0px;"></iframe>
       </div>
       </div>

What doesn't work means? Can't get video ID from that function or what? In case you can't get video ID from that function then you need to post the code in that function not this one. — vee
– vee, Commented Jan 14, 2023 at 17:25
Hi, sorry for my bad wording. I have revised my question again. — Heisenberg
– Heisenberg, Commented Jan 14, 2023 at 18:05
Related: PHP regex : How to use captured groups in a function and Call Function PREG_REPLACE and How to refactor a function to use preg_replace_callback() with multiple regex strings and replacement strings in multiple arrays? — mickmackusa
– mickmackusa ♦, Commented Oct 21, 2023 at 12:02

Chris Haas · Accepted Answer · 2023-01-14 18:08:40Z

The standard preg_replace doesn't allow you to execute a function on the replacement values, you can only pass literals with replacement placeholders.

You can get around this in multiple ways, however preg_replace_callback with that specific case might be the easiest to read. In the following code I also took the liberty of combining your first three shortcodes into a single one that uses a back-reference. I also removed captures that you we're using (for no good reason except to help me debug).

<?php

var_dump(bbc2html('[b]stuff[/b][youtube]abc[/youtube]'));

function getYoutubeVideoID($id)
{
    // Just a demo function
    return '!!'.$id.'!!';
}

function bbc2html($content)
{
    $search = '/\[(b|u|i)]([^\[]*)\[\/\1]/';
    $replace = '<$1>$2</$1>';

    $newContent = preg_replace($search, $replace, $content);

    return preg_replace_callback(
        '/\[youtube](.*?)\[\/youtube]/',
        static function ($matches) {
            return '<div class="video_wrapper card border-0 shadow text-center m-2" style="width: 25rem;">
      <div class="video_trigger card-body" data-source="'.getYoutubeVideoID($matches[1]).'" data-type="youtube">
          <p class="text-center">As soon as you click on the button you accept that cookies from YouTube to be loaded</p>
          <input type="button" class="btn btn-success" value="OK">
          <a href="$2" type="button" target="_blank" class="btn btn-danger"><i class="fa-brands fa-youtube pe-1"></i>direct link</a>
      </div>
      <div class="video_layer" style="display: none;">
          <iframe src="" border="0" data-scaling="true" data-format="16:9" style="height: 0px;"></iframe>
      </div>
      </div>';
        },
        $newContent
    );
}

Demo: https://3v4l.org/0hkQj

Just as a minor improvement, you could write the alternatives \[(b|u|i)] as a character class \[([bui])]

Collectives™ on Stack Overflow

pass preg replace variable to function

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related