Re-added ConstraintKeyObject which is consumed from the AbstractSource w/ test (Fixes #3512) #5903

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
3 participants
Member

ralphschindler commented Mar 3, 2014

This re-adds the ConstraintKeyObject that was removed in a PR/addition last year.
#3512 is where this was identified, while no specific Db related issues address this.

@samsonasik samsonasik commented on the diff Mar 3, 2014

library/Zend/Db/Metadata/Object/ConstraintKeyObject.php
@@ -0,0 +1,256 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2012 Zend Technologies USA Inc. (http://www.zend.com)
@samsonasik

samsonasik Mar 3, 2014

Contributor

now 2014

@samsonasik samsonasik commented on the diff Mar 3, 2014

library/Zend/Db/Metadata/Object/ConstraintKeyObject.php
@@ -0,0 +1,256 @@
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2012 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license http://framework.zend.com/license/new-bsd New BSD License
+ * @package Zend_Db
@samsonasik

samsonasik Mar 3, 2014

Contributor

@Package should be removed

ralphschindler added the Db label Mar 3, 2014

ralphschindler added this to the 2.2.6 milestone Mar 3, 2014

@samsonasik samsonasik commented on the diff Mar 3, 2014

library/Zend/Db/Metadata/Object/ConstraintKeyObject.php
+<?php
+/**
+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2012 Zend Technologies USA Inc. (http://www.zend.com)
+ * @license http://framework.zend.com/license/new-bsd New BSD License
+ * @package Zend_Db
+ */
+
+namespace Zend\Db\Metadata\Object;
+
+/**
+ * @category Zend
+ * @package Zend_Db
+ * @subpackage Metadata
@samsonasik

samsonasik Mar 3, 2014

Contributor

@category @Package @subpackage should be removed

@samsonasik samsonasik commented on the diff Mar 3, 2014

tests/ZendTest/Db/Metadata/Source/AbstractSourceTest.php
@@ -0,0 +1,63 @@
+<?php
+
@samsonasik

samsonasik Mar 3, 2014

Contributor

here should be a license block

@weierophinney weierophinney added a commit that referenced this pull request Mar 3, 2014

@weierophinney weierophinney [#5903] CS fixes
- License and class docblocks
010fb65

@weierophinney weierophinney added a commit that referenced this pull request Mar 3, 2014

@weierophinney weierophinney Merge branch 'hotfix/5903' into develop
Forward port #5903
23c7c4c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment