Recording from June 19th 2015 Drupal 8 critical issues discussion
This was our fourth critical issues discussion meeting to be publicly recorded in a row. (See all prior recordings). This time to make discussions easier to follow for all of us, we switched to #drupal-contribute in IRC to post links, so those following real time can follow the links and we can just paste the meeting log here as well. Here is the recording of the meeting from today in the hope that it helps more than just those who were on the meeting:
Unfortunately not all people invited made it this time. 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:07am] plach: <a href="https://www.drupal.org/node/2478459" title="https://www.drupal.org/node/2478459">https://www.drupal.org/node/2478459</a><br>[11:07am] Druplicon: <a href="https://www.drupal.org/node/2478459" title="https://www.drupal.org/node/2478459">https://www.drupal.org/node/2478459</a> => FieldItemInterface methods are only invoked for SQL storage and are inconsistent with hooks [#2478459] => 93 comments, 19 IRC mentions<br>[11:07am] dawehner: <a href="https://www.drupal.org/node/2500527" title="https://www.drupal.org/node/2500527">https://www.drupal.org/node/2500527</a><br>[11:07am] Druplicon: <a href="https://www.drupal.org/node/2500527" title="https://www.drupal.org/node/2500527">https://www.drupal.org/node/2500527</a> => Rewrite \Drupal\file\Controller\FileWidgetAjaxController::upload() to not rely on form cache [#2500527] => 34 comments, 6 IRC mentions<br>[11:08am] plach: <a href="https://www.drupal.org/node/2453153" title="https://www.drupal.org/node/2453153">https://www.drupal.org/node/2453153</a><br>[11:08am] Druplicon: <a href="https://www.drupal.org/node/2453153" title="https://www.drupal.org/node/2453153">https://www.drupal.org/node/2453153</a> => Node revisions cannot be reverted per translation [#2453153] => 107 comments, 31 IRC mentions<br>[11:09am] jibran: <a href="https://www.drupal.org/node/2263569#comment-10039344" title="https://www.drupal.org/node/2263569#comment-10039344">https://www.drupal.org/node/2263569#comment-10039344</a><br>[11:10am] Druplicon: <a href="https://www.drupal.org/node/2263569" title="https://www.drupal.org/node/2263569">https://www.drupal.org/node/2263569</a> => Bypass form caching by default for forms using #ajax. [#2263569] => 219 comments, 35 IRC mentions<br>[11:11am] Fabianx-screen: <a href="https://www.drupal.org/node/2354889" title="https://www.drupal.org/node/2354889">https://www.drupal.org/node/2354889</a><br>[11:11am] Druplicon: <a href="https://www.drupal.org/node/2354889" title="https://www.drupal.org/node/2354889">https://www.drupal.org/node/2354889</a> => Make block context faster by removing onBlock event and replace it with loading from a BlockContextManager [#2354889] => 66 comments, 13 IRC mentions<br>[11:11am] WimLeers: <a href="https://www.drupal.org/node/2375695" title="https://www.drupal.org/node/2375695">https://www.drupal.org/node/2375695</a><br>[11:11am] Druplicon: <a href="https://www.drupal.org/node/2375695" title="https://www.drupal.org/node/2375695">https://www.drupal.org/node/2375695</a> => Condition plugins should provide cache contexts AND cacheability metadata needs to be exposed [#2375695] => 75 comments, 25 IRC mentions<br>[11:13am] GaborHojtsy: Fabianx-screen is talking about <a href="https://www.drupal.org/node/2354889" title="https://www.drupal.org/node/2354889">https://www.drupal.org/node/2354889</a><br>[11:13am] Druplicon: <a href="https://www.drupal.org/node/2354889" title="https://www.drupal.org/node/2354889">https://www.drupal.org/node/2354889</a> => Make block context faster by removing onBlock event and replace it with loading from a BlockContextManager [#2354889] => 66 comments, 14 IRC mentions<br>[11:14am] WimLeers: No, he was talking about <a href="https://www.drupal.org/node/2501989" title="https://www.drupal.org/node/2501989">https://www.drupal.org/node/2501989</a><br>[11:14am] Druplicon: <a href="https://www.drupal.org/node/2501989" title="https://www.drupal.org/node/2501989">https://www.drupal.org/node/2501989</a> => [meta] Page Cache Performance [#2501989] => 24 comments, 5 IRC mentions<br>[11:14am] WimLeers: (i.e. the very first part of what he said)<br>[11:14am] GaborHojtsy: (I directly copied the link he posted in hangouts :D)<br>[11:14am] WimLeers: lol ok :P<br>[11:16am] WimLeers: <a href="https://www.drupal.org/node/2429287" title="https://www.drupal.org/node/2429287">https://www.drupal.org/node/2429287</a><br>[11:16am] Druplicon: <a href="https://www.drupal.org/node/2429287" title="https://www.drupal.org/node/2429287">https://www.drupal.org/node/2429287</a> => [meta] Finalize the cache contexts API & DX/usage, enable a leap forward in performance [#2429287] => 102 comments, 7 IRC mentions<br>[11:17am] WimLeers: <a href="https://www.drupal.org/node/2450993" title="https://www.drupal.org/node/2450993">https://www.drupal.org/node/2450993</a><br>[11:17am] Druplicon: <a href="https://www.drupal.org/node/2450993" title="https://www.drupal.org/node/2450993">https://www.drupal.org/node/2450993</a> => Rendered Cache Metadata created during the main controller request gets lost [#2450993] => 35 comments, 14 IRC mentions<br>[11:18am] larowlan: GaborHojtsy: still working sorry, sent apology to dawehne_r this morning with my update<br>[11:18am] GaborHojtsy: larowlan: yeah jibran relayed that :)<br>[11:19am] GaborHojtsy: <a href="https://www.drupal.org/node/2495179" title="https://www.drupal.org/node/2495179">https://www.drupal.org/node/2495179</a><br>[11:19am] Druplicon: <a href="https://www.drupal.org/node/2495179" title="https://www.drupal.org/node/2495179">https://www.drupal.org/node/2495179</a> => Twig placeholder filter should not map to raw filter [#2495179] => 53 comments, 7 IRC mentions<br>[11:20am] GaborHojtsy: <a href="https://www.drupal.org/node/2487972" title="https://www.drupal.org/node/2487972">https://www.drupal.org/node/2487972</a><br>[11:20am] Druplicon: <a href="https://www.drupal.org/node/2487972" title="https://www.drupal.org/node/2487972">https://www.drupal.org/node/2487972</a> => [META] Results of testing localize.drupal.org on Drupal 7 in June 2015 [#2487972] => 18 comments, 5 IRC mentions<br>[11:21am] jibran: <a href="https://www.drupal.org/node/2453153" title="https://www.drupal.org/node/2453153">https://www.drupal.org/node/2453153</a><br>[11:21am] Druplicon: <a href="https://www.drupal.org/node/2453153" title="https://www.drupal.org/node/2453153">https://www.drupal.org/node/2453153</a> => Node revisions cannot be reverted per translation [#2453153] => 107 comments, 32 IRC mentions<br>[11:31am] larowlan: jibran++<br>[11:31am] larowlan: GaborHojtsy++<br>[11:31am] GaborHojtsy: Fabianx-screen: what’s the issue link?<br>[11:33am] jibran: <a href="https://www.drupal.org/node/2489024" title="https://www.drupal.org/node/2489024">https://www.drupal.org/node/2489024</a><br>[11:33am] dawehner: <a href="https://www.drupal.org/node/2508591" title="https://www.drupal.org/node/2508591">https://www.drupal.org/node/2508591</a><br>[11:33am] Druplicon: <a href="https://www.drupal.org/node/2489024" title="https://www.drupal.org/node/2489024">https://www.drupal.org/node/2489024</a> => Arbitrary code execution via 'trans' extension for dynamic twig templates (when debug output is on) [#2489024] => 18 comments, 7 IRC mentions<br>[11:33am] Druplicon: <a href="https://www.drupal.org/node/2508591" title="https://www.drupal.org/node/2508591">https://www.drupal.org/node/2508591</a> => Move Drupal into subdirectory and get external dependencies/libraries out of the web-accessible path [#2508591] => 8 comments, 3 IRC mentions<br>[11:42am] dawehner: <a href="https://www.drupal.org/node/2508654#comment-10039315" title="https://www.drupal.org/node/2508654#comment-10039315">https://www.drupal.org/node/2508654#comment-10039315</a><br>[11:42am] Druplicon: <a href="https://www.drupal.org/node/2508654" title="https://www.drupal.org/node/2508654">https://www.drupal.org/node/2508654</a> => File inclusion in transliteration service [#2508654] => 17 comments, 2 IRC mentions<br>[11:43am] GaborHojtsy: dawehner: that one yeah<br>[11:43am] GaborHojtsy: <a href="https://www.drupal.org/drupal8-security-bounty" title="https://www.drupal.org/drupal8-security-bounty">https://www.drupal.org/drupal8-security-bounty</a> running for 2 more months<br>[11:43am] jibran: <a href="https://www.drupal.org/node/1305882" title="https://www.drupal.org/node/1305882">https://www.drupal.org/node/1305882</a><br>[11:43am] Druplicon: <a href="https://www.drupal.org/node/1305882" title="https://www.drupal.org/node/1305882">https://www.drupal.org/node/1305882</a> => drupal_html_id() considered harmful; remove ajax_html_ids to use GET (not POST) AJAX requests [#1305882] => 153 comments, 22 IRC mentions<br>[11:48am] dawehner: <a href="https://www.drupal.org/node/2500523" title="https://www.drupal.org/node/2500523">https://www.drupal.org/node/2500523</a><br>[11:48am] Druplicon: <a href="https://www.drupal.org/node/2500523" title="https://www.drupal.org/node/2500523">https://www.drupal.org/node/2500523</a> => Rewrite views_ui_add_ajax_trigger() to not rely on /system/ajax. [#2500523] => 6 comments, 2 IRC mentions<br>