Added composite hydrator naming strategy #6367

ojhaujjwal · 2014-06-11T15:45:49Z

This naming strategy allows us to use different naming strategy for different property.

samsonasik · 2014-06-12T09:10:46Z

library/Zend/Stdlib/Hydrator/NamingStrategy/CompositeNamingStrategy.php

+     *
+     * @param string $name The name of the naming strategy to get.
+     *
+     * @throws \Zend\Stdlib\Exception\InvalidArgumentException


@throws Exception\InvalidArgumentException

malukenho · 2014-10-10T19:34:28Z

I think this PR need to be rebased.

adamlundrigan · 2014-11-17T01:23:31Z

@ojhaujjwal could you add a blurb to the documentation about how to use this?

Ocramius · 2014-12-17T13:28:41Z

library/Zend/Stdlib/Hydrator/NamingStrategy/CompositeNamingStrategy.php

+     * @throws Exception\InvalidArgumentException
+     * @return NamingStrategyInterface
+     */
+    public function get($name)


I would get rid of all these getters/setters as per discussion on the other PRs :-)

Yes. I will!

Ocramius · 2014-12-28T23:10:59Z

Re-checked this: needs to be refactored in following API IMO:

final class CompositeNamingStrategy
{
    public function __construct(array $strategies, NamingStrategyInterface $defaultStrategy = null) { /* ... */ }

    public function hydrate(...) { /**/ }
    public function extract(...) { /**/ }
}

samsonasik · 2015-01-01T15:10:18Z

library/Zend/Stdlib/Hydrator/NamingStrategy/CompositeNamingStrategy.php

+ * Zend Framework (http://framework.zend.com/)
+ *
+ * @link      http://github.com/zendframework/zf2 for the canonical source repository
+ * @copyright Copyright (c) 2005-2014 Zend Technologies USA Inc. (http://www.zend.com)


ojhaujjwal · 2015-01-03T03:55:05Z

@Ocramius Done!

… `$defaultNamingStrategy`

…`defaultNamingStrategy` if no strategy was found

…develop Close #6367

Ocramius · 2015-01-03T17:15:42Z

@ojhaujjwal merged, thanks! Please note that I also introduced a new IdentityNamingStrategy to simplify this logic on merge.

develop 688f5c1

ojhaujjwal · 2015-01-04T08:36:15Z

@Ocramius 👍

…sed in `$strategies`

…amingStrategy`

…gy` as per specification

… as a default value for the `$defaultNamingStrategy`

…llable

… logic by always using the `defaultNamingStrategy` if no strategy was found

… class)

…ator-naming-strategy' into develop Close zendframework/zendframework#6367

samsonasik reviewed Jun 12, 2014
View reviewed changes

ezimuel added the enhancement label Aug 6, 2014

ezimuel added this to the 2.4.0 milestone Aug 6, 2014

ezimuel added the Stdlib label Aug 6, 2014

EvanDotPro force-pushed the master branch from f77f90b to 21a17e5 Compare September 28, 2014 00:34

Ocramius reviewed Dec 17, 2014
View reviewed changes

Ocramius added the awaiting author updates label Dec 17, 2014

Ocramius self-assigned this Dec 17, 2014

Ocramius added awaiting author updates and removed awaiting author updates labels Dec 28, 2014

samsonasik reviewed Jan 1, 2015
View reviewed changes

ojhaujjwal force-pushed the feature/composite-hydrator-naming-strategy branch 2 times, most recently from 9aa4ea2 to b3ac9fa Compare January 3, 2015 03:32

Added composite hydrator naming strategy

76dc234

ojhaujjwal force-pushed the feature/composite-hydrator-naming-strategy branch from b3ac9fa to 76dc234 Compare January 3, 2015 03:33

5.3 compatibility

53d7637

Ocramius removed the awaiting author updates label Jan 3, 2015

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Using array_map to loop over passed in $strategies

3a39671

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Removing dead code

7c9f2d1

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Clarifying on parameters to be used

01a713c

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Specification for a new IdentityNamingStrategy

ed2864e

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Implementing IdentityNamingStrategy as per specification

668839f

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Using the IdentityNamingStrategy as a default value for the…

2dfa844

… `$defaultNamingStrategy`

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - $defaultNamingStrategy is not nullable

764249d

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Simplified hydrate and extract logic by always using the …

1eb84a7

…`defaultNamingStrategy` if no strategy was found

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Removing dead code

f893b40

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Adding IDE-hint in test code (mock class)

665c7c6

Ocramius added a commit that referenced this pull request Jan 3, 2015

#6367 - Adding coverage annotations

363b802

Ocramius added a commit that referenced this pull request Jan 3, 2015

Merge branch 'feature/#6367-composite-hydrator-naming-strategy' into …

688f5c1

…develop Close #6367

Ocramius closed this Jan 3, 2015

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Using array_map to loop over pas…

6ae94de

…sed in `$strategies`

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Removing dead code

25cef45

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Clarifying on parameters to be used

b880582

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Specification for a new `IdentityN…

2158a51

…amingStrategy`

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Implementing `IdentityNamingStrate…

98306f1

…gy` as per specification

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Using the IdentityNamingStrategy…

8e4c556

… as a default value for the `$defaultNamingStrategy`

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - $defaultNamingStrategy is not nu…

367af8e

…llable

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Simplified hydrate and extract…

b1ab519

… logic by always using the `defaultNamingStrategy` if no strategy was found

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Removing dead code

7d7c3c1

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Adding IDE-hint in test code (mock…

dd05a5e

… class)

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

zendframework/zendframework#6367 - Adding coverage annotations

c778527

gianarb pushed a commit to zendframework/zend-stdlib that referenced this pull request May 15, 2015

Merge branch 'feature/zendframework/zendframework#6367-composite-hydr…

cde48f9

…ator-naming-strategy' into develop Close zendframework/zendframework#6367

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Added composite hydrator naming strategy #6367

Added composite hydrator naming strategy #6367

ojhaujjwal commented Jun 11, 2014

samsonasik Jun 12, 2014

ojhaujjwal Jun 12, 2014

malukenho commented Oct 10, 2014

adamlundrigan commented Nov 17, 2014

Ocramius Dec 17, 2014

ojhaujjwal Dec 17, 2014

Ocramius commented Dec 28, 2014

samsonasik Jan 1, 2015

ojhaujjwal commented Jan 3, 2015

Ocramius commented Jan 3, 2015

ojhaujjwal commented Jan 4, 2015

Added composite hydrator naming strategy #6367

Added composite hydrator naming strategy #6367

Conversation

ojhaujjwal commented Jun 11, 2014

samsonasik Jun 12, 2014

Choose a reason for hiding this comment

ojhaujjwal Jun 12, 2014

Choose a reason for hiding this comment

malukenho commented Oct 10, 2014

adamlundrigan commented Nov 17, 2014

Ocramius Dec 17, 2014

Choose a reason for hiding this comment

ojhaujjwal Dec 17, 2014

Choose a reason for hiding this comment

Ocramius commented Dec 28, 2014

samsonasik Jan 1, 2015

Choose a reason for hiding this comment

ojhaujjwal commented Jan 3, 2015

Ocramius commented Jan 3, 2015

ojhaujjwal commented Jan 4, 2015