search

ifsnop / mysqldump-php

PHP version of mysqldump cli that comes with MySQL
https://github.com/ifsnop/mysqldump-php
GNU General Public License v3.0
1.25k stars 300 forks source link

Improve the library extensibility #199

Closed Luc45 closed 4 years ago

Luc45 commented 4 years ago

I've recently developed a small library that depends on this one. I had to use Reflection to get some stuff done that could be easily prevented if some methods and properties were marked as protected. This PR makes a few methods and properties protected, instead of private, and exposes the default settings on a constant, instead of having it in a local variable in the constructor.

The general goal of this PR is to improve extensibility of this library.

Thanks.

Luc45 commented 4 years ago

Closing this in favor of https://github.com/ifsnop/mysqldump-php/pull/200, since that PR is opened from a branch (master <- some/branch, not master <- master).