[ https://issues.apache.org/jira/browse/CAY-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14100543#comment-14100543 ]
ASF GitHub Bot commented on CAY-1942: ------------------------------------- GitHub user AlexKolonitsky opened a pull request: https://github.com/apache/cayenne/pull/2 CAY-1942 Refactoring of NamedObjectFactory and NamingStrategy * Change enums first letter to lower case * Rename SmartNameGenerator to DefaultNameGenerator; BasicNameGenerator to LegacyNamegenerator, You can merge this pull request into a Git repository by running: $ git pull https://github.com/AlexKolonitsky/cayenne master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cayenne/pull/2.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2 ---- commit 90de78de01b25c2d62bfe7e431e10aa12cb07d15 Author: alexkolonitsky <alex.kolonit...@gmail.com> Date: 2014-08-18T11:18:54Z CAY-1942 Refactoring of NamedObjectFactory and NamingStrategy * Change enums first letter to lower case * Rename SmartNameGenerator to DefaultNameGenerator; BasicNameGenerator to LegacyNamegenerator, ---- > Refactoring of NamedObjectFactory and NamingStrategy > ---------------------------------------------------- > > Key: CAY-1942 > URL: https://issues.apache.org/jira/browse/CAY-1942 > Project: Cayenne > Issue Type: Task > Reporter: Andrus Adamchik > Assignee: Andrus Adamchik > Fix For: 3.2M2 > > > We have a GitHub pull request at https://github.com/apache/cayenne/pull/1 > that refactors NamedObjectFactory and NamingStrategy into something more > consistent and manageable. This should allow us to fix a few naming bugs, and > move on to cdbimport improvements. > The pull request is submitted by Alex Kolonitsky. I have a copy of his CLA, > already submitted to Apache secretary. > Most visible API change: > NamingStrategy -> ObjectNameGenerator -- This message was sent by Atlassian JIRA (v6.2#6252)