When no WOTD is set, the WOTD feed view crashes

The website encountered an unexpected error. Please try again later.<br /><em class="placeholder">Error</em>: Attempt to modify property &quot;_entity&quot; on null in <em class="placeholder">Drupal\g2\WOTD-&gt;viewsPreRender()</em> (line <em class="placeholder">261</em> of <em class="placeholder">modules/contrib/g2/src/WOTD.php</em>). <pre class="backtrace">Drupal\g2\WOTD-&gt;viewsPreRender(Object) (Line: 154)
g2_views_pre_render(Object)
call_user_func_array(Object, Array) (Line: 426)
Drupal\Core\Extension\ModuleHandler-&gt;Drupal\Core\Extension\{closure}(Object, &#039;g2&#039;) (Line: 405)
Drupal\Core\Extension\ModuleHandler-&gt;invokeAllWith(&#039;views_pre_render&#039;, Object) (Line: 433)
Drupal\Core\Extension\ModuleHandler-&gt;invokeAll(&#039;views_pre_render&#039;, Array) (Line: 1528)
Drupal\views\ViewExecutable-&gt;render() (Line: 128)
Drupal\views\Plugin\views\display\Feed-&gt;execute() (Line: 1635)
Drupal\views\ViewExecutable-&gt;executeDisplay(&#039;feed_1&#039;, Array) (Line: 81)
Drupal\views\Element\View::preRenderViewElement(Array)
...snip...

Comments & Activities

Issue created by @fgm
Status changed to Needs review over 1 year ago6:04pm 30 June 2023
Open in Jenkins → Open on Drupal.org →
Core: 9.5.x + Environment: PHP 8.1 & MariaDB 10.3.22
last update over 1 year ago
52 pass
Comment over 1 year ago →
🇫🇷France fgm Paris, France
Patch with a test that fails without the fix and succeeds with the fix.
Comment over 1 year ago →
System Message

fgm → committed 9a4547f2 on 8.x-1.x
Issue #3371672 by fgm: fix WOTD feed on empty glossaries
Status changed to Fixed over 1 year ago7:14pm 30 June 2023
Comment over 1 year ago →
🇫🇷France fgm Paris, France
Merged to current dev HEAD.
Status changed to Postponed: needs info over 1 year ago5:27pm 9 July 2023
Comment over 1 year ago →
🇫🇷France fgm Paris, France
Appears to be broken on D10 on Gitlab CI. Needs investigation.
Status changed to Fixed over 1 year ago8:19pm 9 July 2023
Comment over 1 year ago →
🇫🇷France fgm Paris, France
That is actually an unrelated issue, similar to ✨ Redirect g2/initial without an initial to the G2 main page Fixed .
Comment over 1 year ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

When no WOTD is set, the WOTD feed view crashes

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments & Activities