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

Fix deprecations #129

Merged
merged 5 commits into from Feb 24, 2020
Merged

Fix deprecations #129

merged 5 commits into from Feb 24, 2020

Conversation

freezy-sk
Copy link
Contributor

@freezy-sk freezy-sk commented Feb 21, 2020

Removed unused symfony/templating.
Set minimum version of doctrine/persistence to 1.3.4 (doctrine/persistence#87).
doctrine/orm is forced to 2.6 by doctrine/doctrine-bundle so 2.6.3 makes sense as lowest version (doctrine/orm#7307).

@curry684
Copy link
Member

Good finds, thanks!

@curry684 curry684 merged commit f788225 into omines:master Feb 24, 2020
@freezy-sk freezy-sk deleted the fix-deprecations branch February 24, 2020 11:37
curry684 added a commit that referenced this pull request Apr 28, 2020
* Add server side export

* Add missing dev dependency

* Replace pseudo-type "iterable" by "\Traversable"

To support PHP 7.0

* Replace TranslatorInterface by DataCollectorTranslator

To maintain BC, it is not possible to type hint with the
TranslatorInterface as older versions of Symfony inject "Symfony\Component\Translation\TranslatorInterface"
while newer versions of Symfony inject "Symfony\Contracts\Translation\TranslatorInterface".

* Fix ArgumentCountError (Symfony ^3.4)

* Fix Content Disposition header format

* Do not display the export form

* Excel: render HTML in cells

The 'render' column option is usually use to inject HTML.

* Fix Translator injection

Tests were passing using "Symfony\Component\Translation\DataCollectorTranslator" as type-hint but Symfony may inject "Symfony\Bundle\FrameworkBundle\Translation\Translator".

* Use a Generator instead of a new Iterator

* Using a Generator makes the code more readable.
* Avoid creating an array (iterator_to_array) may be sligtly faster?

* Add functional test "testEmptyDataTable"

* Fix tests

* Preserve DataTableState

Export the entire table but keep current sort and filtering

* Automatically tag exporters

* Add CSV exporter

* Add missing "_dt"

* Fix "DT_RowId" check

* Make dependency on contracts explicit

Will not fix #122 but should prevent similar issues.

* Symfony 3 is not supported, according to the readme.md (#123)

* Symfony 3 is not supported, according to the readme.md

Symfony 3 is not supported, according to the readme.md. 
This updates the documentation to reflect that only Symfony 4.1+ is supported by this bundle

* Update index.html.md

* Option for custom datetime format for creating object (#127)

* Fix failing test by reverting custom WebClient handling

Alternative to #125

* Fix deprecations helper crashing old Symfony versions

* Drop bad testing method on old Symfony frameworks

This is actually obsolete since Symfony Flex, as you can now run Symfony 4.1 with more recent components.

* Apply code style

* Use weak deprecation testing

* Prepare 0.4.1

* Fix deprecations (#129)

* removed unused symfony/templating

* set lowest version of persistence to 1.3.4

* added missing SymfonyTestListener

* raise minimum version of doctrine/orm to 2.6.3

* set phpunit schema location to installed package

* Update translations with script (#130)

* script for updating translation messages from datatables language files

* added czech and slovak languages

* en rebuilt by script

* add missing closing bracket for example (#131)

* Integrate translation update script

* Update README.md

* Restructure tests to improve readability

* Bump nokogiri from 1.10.7 to 1.10.8 in /docs (#133)

Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.10.7 to 1.10.8.
- [Release notes](https://github.com/sparklemotion/nokogiri/releases)
- [Changelog](https://github.com/sparklemotion/nokogiri/blob/master/CHANGELOG.md)
- [Commits](sparklemotion/nokogiri@v1.10.7...v1.10.8)

Signed-off-by: dependabot[bot] <support@github.com>

* Test local translations (#139)

* Upgrade to PHPUnit 8/9

* Update Scrutinizer config

* Force use outdated version of ocramius/package-versions

Breaks both SymfonyInsight and Scrutinizer otherwise.

* Fix SymfonyInsight badge

* Fix SymfonyInsight badge for real

* Javascript improvements (#145)

* JS improvements

* Remove scrollX

* Allow config.options and config.url as functions

* Remove empty line

* Fix export issue when using GET method

* Add server side export

* Add missing dev dependency

* Replace pseudo-type "iterable" by "\Traversable"

To support PHP 7.0

* Replace TranslatorInterface by DataCollectorTranslator

To maintain BC, it is not possible to type hint with the
TranslatorInterface as older versions of Symfony inject "Symfony\Component\Translation\TranslatorInterface"
while newer versions of Symfony inject "Symfony\Contracts\Translation\TranslatorInterface".

* Fix ArgumentCountError (Symfony ^3.4)

* Fix Content Disposition header format

* Do not display the export form

* Excel: render HTML in cells

The 'render' column option is usually use to inject HTML.

* Fix Translator injection

Tests were passing using "Symfony\Component\Translation\DataCollectorTranslator" as type-hint but Symfony may inject "Symfony\Bundle\FrameworkBundle\Translation\Translator".

* Use a Generator instead of a new Iterator

* Using a Generator makes the code more readable.
* Avoid creating an array (iterator_to_array) may be sligtly faster?

* Add functional test "testEmptyDataTable"

* Fix tests

* Preserve DataTableState

Export the entire table but keep current sort and filtering

* Automatically tag exporters

* Add CSV exporter

* Add missing "_dt"

* Fix "DT_RowId" check

* Fix export issue when using GET method

* Fix tests

Co-authored-by: Niels Keurentjes <niels.keurentjes@omines.com>
Co-authored-by: Spyros Sakellaropoulos <spyridonas@users.noreply.github.com>
Co-authored-by: freezy <freezy-sk@users.noreply.github.com>
Co-authored-by: rwkt <37826694+rwkt@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Daniel Gorgan <danut007ro@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants