Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

setMode, getMode, setLang are not probably called. #20

Open
alhoqbani opened this issue Jul 11, 2017 · 0 comments
Open

setMode, getMode, setLang are not probably called. #20

alhoqbani opened this issue Jul 11, 2017 · 0 comments
Labels
bug class: Arabic class: Date class: Query

Comments

@alhoqbani
Copy link
Owner

When any of these methods is called (setMode, getMode & setLang) , line 255 will try to reload their respective classes. This is because different classes have methods with the same name (Date, Query, Soundex, CompressStr).

However, this leads to invoking the wrong class. For example, calling Query::setMode, will invoke Date::setMode because Date comes before Query in the config.xml
alhoqbani added a commit that referenced this issue Jul 11, 2017
@alhoqbani
Add test set and get mode.
b01a433 
The test is falling because of the conflict with the methodsof the same
name on the date class. See #20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug class: Arabic class: Date class: Query
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alhoqbani