Fix Undefined property: stdClass:: in common.inc#41
Fix Undefined property: stdClass:: in common.inc#41claudiu-cristea wants to merge 1 commit intod6lts:6.xfrom
Conversation
|
I can't decide if we generally want this. It does preserve current functionality, but it also feels like it could cover up deeper issues. What happens if some buggy module passes a language string in $options['language'], rather than an object? You'd want to see the notice rather than have "" being used. (Though stock Drupal unfortunately suppresses Notices...) That's not your situation, where $options['language'] contains an incomplete object... I can't really think of when that would happen. Will keep a look out while running a multilanguage D6 site through simpletest. |
|
@rmuit Did you arrive at any further conclusions on this issue with the multilanguage D6 site? |
|
I have now finished testing some multilanguage sites on PHP8.1. (With the simpletest patch applied, otherwise notices are ignored.) I have not encountered any notices in url(). So I'm strengthened in my hesitance to apply this. @izmeez did you encounter that notice? |
I'm getting this notice: