Sergei Predvoditelev

Results 264 comments of Sergei Predvoditelev

> If no colums passed then means columns have order as in table schema But why need columns in this case? Isn't that the equivalent to way 3?

`params.php` - basicaly a place for service configuration. Application params configure `ApplicationParameter` class, that use directly for get params. Perhaps will be more convenient use separate `application-params.php` for it.

Actually, this method don't allow mark command for run in transaction with default isolation level. `->requireTransaction()` don't run command in transaction. `CommandInterface` do not contain `requireTransaction` method. Suggest remove `requireTransaction`...

@darkdef @terabytesoftw What do you think? Can we remove `requireTransaction()` method?

> I can make a console configuration wizard May be should it be separate console utility in separate repo? > Is [it a duplicate](https://github.com/yiisoft/yii-dev-panel/issues/65)? No. It is issue for Yii...

There is problem when infinite scroll usage with AJAX. If loading property always false, then everything work good, but if use loading (true before start ajax and false after finish),...

@hiscaler You should add optional routes to `withOptionalPatterns()` method. For example: ```php Authentication::class => [ 'class' => Authentication::class, '__construct()' => [ 'authenticationFailureHandler' => Reference::to(PassportRequestErrorHandler::class), ], 'withOptionalPatterns()' => [ 'optional' =>...

@hiscaler you want make optional authentication for GET request only and otherwise required (POST, DELETE, etc.)?

Need add ability to specify request type in optional patterns.

Another idea: leave in `Authentication` middleware authentication process only and create new middleawre that will be throw exception or return response "No access".