Incorrect access level on $message property in class getid3_exception

[yrpwayne]

In the file: wp-includes/ID3/getip3.php

on line: 1795

class getid3_exception extends Exception
{
	public $message;
}

Should be

class getid3_exception extends Exception
{
	protected $message;
}

[obenland]

Given that $message is a protected property in the original Exception class, I don't see a reason for the existence of getid3_exception other than changing the access level on that property. What do you think?

[yrpwayne]

Yes well, that seems to be what's happening. I assume it was done for testing because I cannot find any use of the exposed property or reason for it.

Also, you can call getMessage() if you need to read the contents and you should create a new instance if you need to modify it.

Looking with a clearer head in the morning I see it is a 3rd party library, so I’ll move this issue over there and see if there’s a good reason.

[hellofromTonya]

The code in question is part of an external library. As @yrpwayne notes,

Looking with a clearer head in the morning I see it is a 3rd party library, so I’ll move this issue over there and see if there’s a good reason.

Closing to be/was reported upstream.