Should not require yii-web about yii-core HOT 11 CLOSED

@machour i think we gonna have to change it to an abstract class and remove Yii-web dep and made something like Application class.
I found another thing that I'm working on - it uses request object, but logically it must be available only of Yii-web, not CLI.

from yii-core.

@machour at this moment I have PSR-15 and Error processing in my backlog in order to at least partially support RoadRunner.
If no one will take care of it before I finish with other things I can look into it.

from yii-core.

As for me, there are still things in core that should be moved out :)

from yii-core.

Actually its a good question why Pagination is in core package)

from yii-core.

API is web.

from yii-core.

I guess it's still here because Pagination could be used either from Web, or API, or even CLI for that matter.
Putting it into yiisoft/yii-dataviewcould make sense for the "web" part, but not the others.. 🤔

from yii-core.

To me it makes sense.

from yii-core.

@SilverFire, @hiqsol what do you think?

from yii-core.

@Alex-Bond would you like to take care of this?

from yii-core.

Hi !

Currently, src/data/ files require yii-web:
Putting it into yiisoft/yii-dataview

Is it about ?

1. to move
   https://github.com/yiisoft/yii-core/tree/master/src/data/*.php
   into
   https://github.com/yiisoft/yii-dataview/tree/MOVED_YII_WEB_WIDGETS/src
   (MOVED_YII_WEB_WIDGETS - branch name)

2. to remove php files and 'data' folder
   https://github.com/yiisoft/yii-core/tree/MOVED_YII_WEB_WIDGETS/src/data/*.php

master -> MOVED_YII_WEB_WIDGETS

from yii-core.

Done by now. data and data-widgets are extracted into separate packages.

from yii-core.