Improvements for get_file_data() function

[Tkama]

It seams as a bug or as imperfection at least.

I try to explain in example.

Assume we have such comments in file:

<?php
/*
Plugin Name: My plug
Version: 1.0
*/

// and trying to get file data like:
$data = get_file_data( __FILE__, ['ver'=>'Version'] );
echo $data['ver']; //> 1.0

This is works as we need.

BUT if we change comments a little, the get_file_data() becomes broken:

<?php
/*
Plugin Name : My plug
Version     : 1.0
*/

// and trying to get file data like^
$data = get_file_data( __FILE__, ['ver'=>'Version'] );
echo $data['ver']; //> ''

All we do just add spaces before : to make comments more readable...

-

Solution

I think it's better to improve regular expression of the get_file_data() and add [\t ]* before :

<?php

if ( preg_match( '/^[ \t\/*#@]*' . preg_quote( $regex, '/' ) . ':(.*)$/mi', $file_data, $match ) && $match[1] )

to 

if ( preg_match( '/^[ \t\/*#@]*' . preg_quote( $regex, '/' ) . '[\t ]*:(.*)$/mi', $file_data, $match ) && $match[1] )

[Tkama]

The same applies to function get_plugin_data(), bacause it relies on get_file_data()

[subrataemfluence]

Good catch! :)

[Tkama]

Thanks a lot! :)

[subrataemfluence]

Replying to Tkama:

Thanks a lot! :)

You are welcome:) Would you mind testing the patch and see if it works as desired?
Thank you.

[Tkama]

I already tested code I suggested - it works fine!

But, off course if I will see troubles about this issue and give to know...