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

No deprecation #92

Merged
merged 11 commits into from
Aug 7, 2018
Merged

Conversation

Modicrumb
Copy link
Contributor

I tried my best to make it deprecation free in CakePHP 3.6. As a result it's probably not BC with CakePHP versions less than 3.5. I've also updated the documentation for the new withDownload method.

Fixed deprecation errors, changed unit tests to use newer methods.
@dereuromark
Copy link
Member

Can you resolve the conflict?

@dereuromark
Copy link
Member

Don't forget to also adjust composer then and set proper min version for the framework
3.0 sure aint true anymore.

@Modicrumb
Copy link
Contributor Author

Modicrumb commented Aug 3, 2018

I've resolved the conflict, regardless, tests do not pass because I'm still using the deprecated method type() instead of withType(), because withType() does not properly replace type() for new types. Meaning I am just using type(). Deprecation errors come through due to this though. I wasn't sure if I should just file a separate issue in the cakephp github. But I would need to set up a test to make sure that I'm right about withType not handling new types like type() did.

Oh shnap. It seems tests are actually going through on some places? Maybe all I need to do is run codesniffer on this stuff and I suppose update composer?

changes to composer to disclude non-compatible versions of cake
@codecov
Copy link

codecov bot commented Aug 6, 2018

Codecov Report

Merging #92 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #92   +/-   ##
=========================================
  Coverage     97.63%   97.63%           
  Complexity       55       55           
=========================================
  Files             1        1           
  Lines           127      127           
=========================================
  Hits            124      124           
  Misses            3        3
Impacted Files Coverage Δ Complexity Δ
src/View/CsvView.php 97.63% <ø> (ø) 55 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 705d7d7...74618ed. Read the comment docs.

fixed the missing comma now
Changes to satisfy some PSR standards, for some reason bootstrap.php is
failing in one of the tests, so this is also an attempt to fix that.
3.4 is now the minimum
Now min 3.5 with PHP 5.6
@Modicrumb
Copy link
Contributor Author

I'm baffled at the Fatal error: Call to undefined method Cake\Http\Response::getType() in /home/travis/build/FriendsOfCake/cakephp-csvview/tests/TestCase/View/CsvViewTest.php on line 41

@Modicrumb
Copy link
Contributor Author

Alright so getType is not included until 3.5.5 my bad, will fix this when I get home

@dereuromark
Copy link
Member

I helped.
The Requirements section in README should mention 3.5.5+ then, as well.
Otherwise all good and we can release a minor here, since this is all BC then.

changed requirements in readme
@Modicrumb
Copy link
Contributor Author

Fixed up the readme.

@dereuromark dereuromark merged commit 48d9b97 into FriendsOfCake:master Aug 7, 2018
@dereuromark
Copy link
Member

Now we just need to fix CS.

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