Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deprecate this bundle #280

Closed
core23 opened this issue Dec 19, 2019 · 14 comments
Closed

Deprecate this bundle #280

core23 opened this issue Dec 19, 2019 · 14 comments

Comments

@core23
Copy link
Member

core23 commented Dec 19, 2019

There are good and better maintained replacements on the market:

Refs #182

@core23 core23 added the feature label Dec 19, 2019
@core23
Copy link
Member Author

core23 commented Dec 19, 2019

cc @sonata-project/contributors

@greg0ire
Copy link
Contributor

I'm all for it! There are some Sonata packages that depend on this:

  • sonata-project/admin-bundle
  • sonata-project/ecommerce
  • sonata-project/media-bundle
  • sonata-project/news-bundle
  • sonata-project/timeline-bundle

@core23
Copy link
Member Author

core23 commented Dec 19, 2019

The only thing that we still need is the locale detector that updates the user entity.

@franmomu
Copy link
Member

Trying to get rid of this bundle in TimelineBundle, the problem is that twig/intl-extra has "symfony/intl": "^4.3|^5.0" requirement and we have in Travis one job with symfony/symfony: 3.4.*.

@greg0ire
Copy link
Contributor

I think it's ok to drop 3.4, we are only supposed to support one LTS at a time: https://github.com/sonata-project/SonataIntlBundle/blob/2.x/CONTRIBUTING.md#dependency-changes

@XWB
Copy link
Contributor

XWB commented Jan 6, 2020

@core23 Unfortunately twigphp/intl-extra only provides Twig filters for number and date formatting. AFAIK Symfony has no built in services to achieve something like:

use Sonata\IntlBundle\Templating\Helper\DateTimeHelper;

class MyClass
{
    private $dateTimeHelper;

    public function __construct(DateTimeHelper $dataTimeHelper)
    {
        $this->dateTimeHelper = $dataTimeHelper;
    }

    public function myFunction(string $format)
    {
        // to something with $this->dateTimeHelper
    }
}

@core23
Copy link
Member Author

core23 commented Feb 7, 2020

@core23 Unfortunately twigphp/intl-extra only provides Twig filters for number and date formatting. AFAIK Symfony has no built in services to achieve something like:

use Sonata\IntlBundle\Templating\Helper\DateTimeHelper;

class MyClass
{
    private $dateTimeHelper;

    public function __construct(DateTimeHelper $dataTimeHelper)
    {
        $this->dateTimeHelper = $dataTimeHelper;
    }

    public function myFunction(string $format)
    {
        // to something with $this->dateTimeHelper
    }
}

Maybe we could move from this bundle to the symfony bundle?

@core23 core23 pinned this issue Feb 14, 2020
@VincentLanglet
Copy link
Member

Shouldn't the readme be updated if this repository is deprecated ?

@core23
Copy link
Member Author

core23 commented Mar 2, 2020

Shouldn't the readme be updated if this repository is deprecated ?

Can you do this @VincentLanglet

@VincentLanglet
Copy link
Member

Yes I can @core23
#296

But I also saw that the deprecated packages had this
image

I don't have the right to modify the description.

@greg0ire
Copy link
Contributor

Thanks for pointing it out. Looks like I have the necessary permissions to do this, I just changed the description.

@VincentLanglet
Copy link
Member

Thanks for pointing it out. Looks like I have the necessary permissions to do this, I just changed the description.

Can you do the same for https://github.com/sonata-project/SonataCacheBundle and https://github.com/sonata-project/SonataEasyExtendsBundle

@greg0ire
Copy link
Contributor

Done (TIL the cache bundle is deprecated 😅)

@VincentLanglet
Copy link
Member

Cf #300 (comment)

@VincentLanglet VincentLanglet unpinned this issue May 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants