Skip to content

Update cleaned name to "fix" dynamic dispatch classes

Roy Zwambag requested to merge 396460-clean-dynamic-dispatch-name into master

What does this MR do and why?

A lot of context is in the issue #396460 (closed). We have to rename dynamic dispatch classes like #<Class:0x00007f6ecb77b460>#_app_views_layouts_project_html_haml___3698311611987171681_369660 into something DynamicClass_app_views_layouts_project_html_haml.call. Note that we add .call so that Google Profiler sees we are calling a function, and will merge all "classes" (DynamicClass_app_views_layouts_project_html_haml) together. This is very similar to what we did at !114613 (merged).

We opted to clean everything within the same function, in one long regex. So we look for #<Class:anything>#_anything_haml_123, #, and ::.

How to set up and validate locally

  • Run specs

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Fixes #396460 (closed)

Merge request reports