I was looking at some of the code in this module and noticed that variable declarations had been changed.
For example,
Code:
var $_clashing_mappings = array();
has been changed to
Code:
public $_clashing_mappings = array();
This despite the fact that the original authors comments prior to the declaration are
/**
* Maintains a list of any URI mappings for the Manufacturer which clash with existing mappings.
*
* @var array
* @access protected
*/
Shouldn't that have been changed to
Code:
protected $_clashing_mappings = array();
Bookmarks