fixes #4604 #5379

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@steverhoades
Contributor

steverhoades commented Oct 29, 2013

If class contains constructor addFunction will no longer result in an error. This fixes issue #4604

@@ -493,6 +523,38 @@ public function baz()
}
}
+class Bar

This comment has been minimized.

Show comment Hide comment
@weierophinney

weierophinney Oct 30, 2013

Member

This should be in a separate file. I can do that on merge.

@weierophinney

weierophinney Oct 30, 2013

Member

This should be in a separate file. I can do that on merge.

weierophinney added a commit that referenced this pull request Oct 30, 2013

weierophinney added a commit that referenced this pull request Oct 30, 2013

[#5379] Move test assets to separate files
- Moved classes, functions used by test case to separate files

weierophinney added a commit that referenced this pull request Oct 30, 2013

@ghost ghost assigned weierophinney Oct 30, 2013

weierophinney added a commit to zendframework/zend-json that referenced this pull request May 15, 2015

weierophinney added a commit to zendframework/zend-json that referenced this pull request May 15, 2015

[zendframework/zendframework#5379] Move test assets to separate files
- Moved classes, functions used by test case to separate files

weierophinney added a commit to zendframework/zend-json that referenced this pull request May 15, 2015

weierophinney added a commit to zendframework/zend-json that referenced this pull request May 15, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment