Recording from August 28th 2015 Drupal 8 critical issues discussion
We met again today to discuss critical issues blocking Drupal 8's release (candidate). (See all prior recordings). Here is the recording of the meeting video and chat from today in the hope that it helps more than just those who were on the meeting:
If you also have significant time to work on critical issues in Drupal 8 and we did not include you, let me know as soon as possible.
The meeting log is as follows (all times are CEST real time at the meeting):
<br>[11:08am] GaborHojtsy: <a href="https://www.drupal.org/node/2555183" title="https://www.drupal.org/node/2555183">https://www.drupal.org/node/2555183</a><br>[11:08am] Druplicon: <a href="https://www.drupal.org/node/2555183" title="https://www.drupal.org/node/2555183">https://www.drupal.org/node/2555183</a> => Fix the filled update tests, they are broken [#2555183] => 60 comments, 10 IRC mentions<br>[11:08am] GaborHojtsy: <a href="https://www.drupal.org/node/2555665" title="https://www.drupal.org/node/2555665">https://www.drupal.org/node/2555665</a><br>[11:08am] Druplicon: <a href="https://www.drupal.org/node/2555665" title="https://www.drupal.org/node/2555665">https://www.drupal.org/node/2555665</a> => When index is added for content_translation_uid, the corresponding stored schema definition is not updated [#2555665] => 30 comments, 7 IRC mentions<br>[11:09am] plach: <a href="https://www.drupal.org/node/2542748" title="https://www.drupal.org/node/2542748">https://www.drupal.org/node/2542748</a><br>[11:09am] Druplicon: <a href="https://www.drupal.org/node/2542748" title="https://www.drupal.org/node/2542748">https://www.drupal.org/node/2542748</a> => Automatic entity updates can fail when there is existing content, leaving the site's schema in an unpredictable state [#2542748] => 184 comments, 38 IRC mentions<br>[11:10am] plach: <a href="https://www.drupal.org/node/2558905" title="https://www.drupal.org/node/2558905">https://www.drupal.org/node/2558905</a><br>[11:10am] Druplicon: <a href="https://www.drupal.org/node/2558905" title="https://www.drupal.org/node/2558905">https://www.drupal.org/node/2558905</a> => Content translation module - Information disclosure by insufficient access checking [#2558905] => 9 comments, 3 IRC mentions<br>[11:11am] plach: <a href="https://www.drupal.org/node/2555665" title="https://www.drupal.org/node/2555665">https://www.drupal.org/node/2555665</a><br>[11:11am] Druplicon: <a href="https://www.drupal.org/node/2555665" title="https://www.drupal.org/node/2555665">https://www.drupal.org/node/2555665</a> => When index is added for content_translation_uid, the corresponding stored schema definition is not updated [#2555665] => 30 comments, 8 IRC mentions<br>[11:13am] WimLeers: <a href="https://www.drupal.org/node/2429617#comment-10256775" title="https://www.drupal.org/node/2429617#comment-10256775">https://www.drupal.org/node/2429617#comment-10256775</a><br>[11:13am] Druplicon: <a href="https://www.drupal.org/node/2429617" title="https://www.drupal.org/node/2429617">https://www.drupal.org/node/2429617</a> => Make D8 2x as fast: SmartCache: context-dependent page caching (for *all* users!) [#2429617] => 265 comments, 34 IRC mentions<br>[11:13am] WimLeers: <a href="https://www.drupal.org/node/2556889" title="https://www.drupal.org/node/2556889">https://www.drupal.org/node/2556889</a><br>[11:13am] Druplicon: <a href="https://www.drupal.org/node/2556889" title="https://www.drupal.org/node/2556889">https://www.drupal.org/node/2556889</a> => [policy, no patch] Decide if SmartCache is still in scope for 8.0 and whether remaining risks require additional mitigation [#2556889] => 62 comments, 4 IRC mentions<br>[11:19am] alexpott: xjm: <a href="https://www.drupal.org/node/2558791" title="https://www.drupal.org/node/2558791">https://www.drupal.org/node/2558791</a><br>[11:19am] Druplicon: <a href="https://www.drupal.org/node/2558791" title="https://www.drupal.org/node/2558791">https://www.drupal.org/node/2558791</a> => !placeholder should Xss::adminFilter but not affect safeness [#2558791] => 11 comments, 1 IRC mention<br>[11:20am] alexpott: xjm: your issue might be a duplicate<br>[11:26am] WimLeers: plach: <a href="https://www.drupal.org/node/2558905#comment-10267715" title="https://www.drupal.org/node/2558905#comment-10267715">https://www.drupal.org/node/2558905#comment-10267715</a><br>[11:26am] Druplicon: <a href="https://www.drupal.org/node/2558905" title="https://www.drupal.org/node/2558905">https://www.drupal.org/node/2558905</a> => Content translation module - Information disclosure by insufficient access checking [#2558905] => 9 comments, 4 IRC mentions<br>[11:28am] plach: WimLeers: replied<br>[11:33am] jibran: <a href="https://www.drupal.org/node/2538108" title="https://www.drupal.org/node/2538108">https://www.drupal.org/node/2538108</a><br>[11:33am] Druplicon: <a href="https://www.drupal.org/node/2538108" title="https://www.drupal.org/node/2538108">https://www.drupal.org/node/2538108</a> => Add an API for data value updates to reliably run after data format updates [#2538108] => 19 comments, 4 IRC mentions<br>[11:46am] WimLeers: <a href="https://www.drupal.org/node/2557815#comment-10266477" title="https://www.drupal.org/node/2557815#comment-10266477">https://www.drupal.org/node/2557815#comment-10266477</a><br>[11:46am] Druplicon: <a href="https://www.drupal.org/node/2557815" title="https://www.drupal.org/node/2557815">https://www.drupal.org/node/2557815</a> => Automatically assign node grants cache context in node_query_node_access_alter() [#2557815] => 17 comments, 2 IRC mentions<br>[11:53am] jibran: <a href="https://www.drupal.org/node/2538108" title="https://www.drupal.org/node/2538108">https://www.drupal.org/node/2538108</a><br>[11:53am] Druplicon: <a href="https://www.drupal.org/node/2538108" title="https://www.drupal.org/node/2538108">https://www.drupal.org/node/2538108</a> => Add an API for data value updates to reliably run after data format updates [#2538108] => 19 comments, 5 IRC mentions<br>[11:55am] WimLeers: <a href="https://www.drupal.org/node/2464427" title="https://www.drupal.org/node/2464427">https://www.drupal.org/node/2464427</a><br>[11:55am] Druplicon: <a href="https://www.drupal.org/node/2464427" title="https://www.drupal.org/node/2464427">https://www.drupal.org/node/2464427</a> => Replace CacheablePluginInterface with CacheableDependencyInterface [#2464427] => 176 comments, 27 IRC mentions<br>[11:55am] jibran: <a href="https://www.drupal.org/node/2538108" title="https://www.drupal.org/node/2538108">https://www.drupal.org/node/2538108</a><br>[11:55am] Druplicon: <a href="https://www.drupal.org/node/2538108" title="https://www.drupal.org/node/2538108">https://www.drupal.org/node/2538108</a> => Add an API for data value updates to reliably run after data format updates [#2538108] => 19 comments, 6 IRC mentions<br>[11:56am] xjm: Is that another update path test we need? a test contrib module?<br>[12:05pm] • xjm finds herself wondering if plach hears churchbells again :)<br>[12:05pm] plach: xjm: I'm doing right now :)<br>[12:05pm] xjm: :D<br>[12:13pm] xjm: Given the complexity here, I find myself wondering how on earth it was we ever expected this to work for major version upgrades :) Thank goodness for migrate<br>