Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 8, 2020. It is now read-only.

Fix annotations on zend db#6798

Closed
dennisdegreef wants to merge 4 commits into
zendframework:masterfrom
dennisdegreef:fixAnnotationsOnZendDb
Closed

Fix annotations on zend db#6798
dennisdegreef wants to merge 4 commits into
zendframework:masterfrom
dennisdegreef:fixAnnotationsOnZendDb

Conversation

@dennisdegreef
Copy link
Copy Markdown
Contributor

@dennisdegreef dennisdegreef commented Oct 23, 2014

@dennisdegreef dennisdegreef mentioned this pull request Oct 24, 2014
Merged
@samsonasik
Copy link
Copy Markdown
Contributor

samsonasik commented Dec 2, 2014

please do rebase againts master and fix the conflicts.

@dennisdegreef
Copy link
Copy Markdown
Contributor Author

dennisdegreef commented Dec 2, 2014

Done! :)

@Ocramius
Copy link
Copy Markdown
Member

Ocramius commented Dec 2, 2014

@dennisdegreef commit dennisdegreef/zf2@433a40e should be removed. Rebase instead, merging master into dev branches causes merge/bisect hell.

@dennisdegreef dennisdegreef force-pushed the fixAnnotationsOnZendDb branch from 433a40e to b085ded Compare December 2, 2014 23:50
@dennisdegreef
Copy link
Copy Markdown
Contributor Author

dennisdegreef commented Dec 2, 2014

Sorry, I may have messed this up... I'll try and repatch it on a fresh branch

@Ocramius
Copy link
Copy Markdown
Member

Ocramius commented Dec 2, 2014

@dennisdegreef looking much better now

@Ocramius Ocramius added this to the 2.3.4 milestone Dec 2, 2014
@Ocramius Ocramius self-assigned this Dec 2, 2014
Ocramius
Ocramius reviewed Dec 5, 2014
View reviewed changes
Comment thread library/Zend/Db/Adapter/AdapterInterface.php
Copy link
Copy Markdown
Member

@Ocramius Ocramius Dec 5, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a major BC break and I will have to remove it on merge.

Ocramius added a commit that referenced this pull request Dec 5, 2014
@Ocramius
#6798 - reverting BC break in AdapterInterface (added method cannot…
10c31a3 
… be added in 2.x)
Ocramius added a commit that referenced this pull request Dec 5, 2014
@Ocramius
#6798 - removing unused imports
31b6581
@Ocramius Ocramius closed this in f373802 Dec 5, 2014
Ocramius added a commit that referenced this pull request Dec 5, 2014
@Ocramius
Merge branch 'hotfix/#6798-fix-zend-db-docblock-annotations-as-per-sc…
cbbf68d 
…rutinizer' into develop

Close #6798
Forward port #6798
@Ocramius
Copy link
Copy Markdown
Member

Ocramius commented Dec 5, 2014

@dennisdegreef manually merged, thanks!

master: f373802
develop: cbbf68d

@dennisdegreef
Copy link
Copy Markdown
Contributor Author

dennisdegreef commented Dec 5, 2014

Hi @Ocramius , If the query() method in the interface is a BC break (I can't see why currently, but that may be the time of day), shouldn't the getMock() methods in the tests also remove the 'query' method from the second parameter?

$mockMasterAdapter = $this->getMock(
            'Zend\Db\Adapter\AdapterInterface',
-            array('getDriver', 'getPlatform', 'query')
+           array('getDriver', 'getPlatform')
        );

@Ocramius
Copy link
Copy Markdown
Member

Ocramius commented Dec 5, 2014

It's a bc break because implementing classes not implementing query() will then cause a fatal (non-framework classes).

I didn't see any other failures otherwise: getMock could indeed need some fixing there, if you want to open a different PR :-)

@dennisdegreef dennisdegreef mentioned this pull request Dec 6, 2014
Closed
Ocramius added a commit that referenced this pull request Dec 7, 2014
@Ocramius
#6968 #6798 - minor CS fixes (default values)
1f3943f
Ocramius added a commit that referenced this pull request Dec 7, 2014
@Ocramius
#6968 #6798 - removing explicit method mocking (not required with int…
d208821 
…erfaces)
Ocramius added a commit that referenced this pull request Dec 7, 2014
@Ocramius
#6968 #6798 - removing explicit method mocking (not required with int…
2751a43 
…erfaces)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@Ocramius Ocramius

Labels

Db enhancement

Projects

None yet

Milestone

2.3.4

Development

Successfully merging this pull request may close these issues.

3 participants

@dennisdegreef @samsonasik @Ocramius