Hi,
I found two bugs in this module.
1. in select country section: Didn't showed Hungary option, and the Poland and Netherland options were in wrong rows.
2. The format function did generated error message, because four matches are unnecessary.
I attached the patch, that solve this problem.
Please review, and if it work, you can use it.
Thanks,
Lohn Dániel
| Comment | File | Size | Author |
|---|---|---|---|
| country_settings.patch | 993 bytes | lohndaniel |
Comments
Comment #1
dpovshed commentedHi Lohn,
I tested your patch against latest dev version and it seems fine with me.
The patch fixed exactly these problems as you described, so I vote for this small fix.
Comment #2
g1smd commentedI've taken care of the phone.hu.inc change along with several others within the phone.hu.inc file inside another patch.
Comment #3
slcp commented@g1smd - Please reference the issue where this patch can be found (like this [#issue-number]) and the close this as duplicate if appropriate.
Alternatively you may need to break down your patch into multiple patches to address different issues if it is outside the scope of the issue it is already attached to...just trying to cover all bases :o)
Comment #4
g1smd commentedI'm waiting for the module owner to re-appear before devoting any more time to this module.
I have a large number of patches already prepared but not yet uploaded.
Comment #5
rfsbsbThis issue has been fixed in #1891040: Error when entering the correct value...