Skip to content
This repository has been archived by the owner on Nov 21, 2019. It is now read-only.

'Embded' should be changed to 'Embedded' #2

Merged
1 commit merged into from
Nov 29, 2010
Merged

'Embded' should be changed to 'Embedded' #2

1 commit merged into from
Nov 29, 2010

Conversation

autoferrit
Copy link

Reading some of the methods I saw the use of Embded and at first did not realize it was for the embedded functionality. So i refactored the 'Embded' to 'Embedded' and 'embded' to 'embedded'. Everything was refactored so it should still be working. in your models, the readme states for embedded documents you have to enter:

public function embeddedDocuments( ) {
    return array( 'address' => 'ClientAddress' // property name => embded document class name
    );
}

which should be:

public function embeddedDocuments( ) {
    return array( 'address' => 'ClientAddress' // property name => embded document class name
    );
}

…factored methods using the text 'Embded' to 'Embedded' for easier read and typing
@canni
Copy link
Owner

canni commented Nov 29, 2010

Thanks, i'm not native english speaker, language help would be nice :)

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants