View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0002157 | OXID eShop (all versions) | 4. ------ eShop Core ------- | public | 2010-10-13 11:43 | 2012-12-10 13:42 |
| Reporter | laluelala2 | Assigned To | |||
| Priority | normal | Severity | major | Reproducibility | sometimes |
| Status | resolved | Resolution | no change required | ||
| Product Version | 4.4.3 revision 30016 | ||||
| Target Version | 4.5.2 revision 38481 | Fixed in Version | 4.4.4 revision 30554 | ||
| Summary | 0002157: Captcha not displayed when displaying PHP notices | ||||
| Description | Due to the following code PHP produces a notice message which breaks the display of randomly generated captchas. if ( !isset( $aLanguages[$iLang] ) && is_array( $aLanguages ) ) This code results in following message: Notice: Trying to get property of non-object in XXX/core/oxlang.php on line 541 The array aLanguages is empty for some reasons. Checking if aLanguages is an array is not enough. Please add or change this condition to sizeof($aLanguages) or something like that to sure that the array is filled with at least one entry. The display of this notice results in following PHP warning: Warning: Cannot modify header information - headers already sent... Therefore the captcha cannot be displayed and the form cannot be sent. | ||||
| Steps To Reproduce | This bug cannot be reproduced because at least one language is marked as active and should be in this array. | ||||
| Tags | No tags attached. | ||||
| Theme | Both | ||||
| Browser | All | ||||
| PHP Version | any | ||||
| Database Version | any | ||||
|
|
@developers: please, check from source code side, if such issue exist. |
|
|
Added additional checking if language array in oxlang::validateLanguage() is not empty. |
|
|
Reopened. Priority decreased as the user case when no active language is in eshop is very rare. |
|
|
testes with fronted when no active language is set. work without notices (get default language) in error mode E_ALL |