View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0007037 | OXID eShop (all versions) | 4.07. Source code, Test | public | 2019-09-25 17:01 | 2024-06-26 10:18 |
| Reporter | pbenke | Assigned To | |||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | no change required | ||
| Product Version | 6.1.4 | ||||
| Summary | 0007037: Misspelling in function call | ||||
| Description | Misspelling in function call: File: vendor/oxid-esales/oxideshop-ce/source/Application/Model/Order.php Function: _sendOrderByEmail() Misspelling function calls: - sendOrderEMailToUser => correct would be: sendOrderEmailToUser (small "m") - sendOrderEMailToOwner => correct would be sendOrderEmailToOwner (small "m") | ||||
| Tags | Email, Order | ||||
| Theme | Not defined | ||||
| Browser | Not defined | ||||
| PHP Version | 7.1 | ||||
| Database Version | Not defined | ||||
|
|
That is not really correct. There are 2 possible spellings in english: email and e-mail The method names are derived from the second variant (the hypen is of course no valid character in method names). I do agree that the spelling should be identical in all places, but it would cause more mayhem to fix that for single calls than it benefits anyone. Would also only possible in a major release |
