Skip to content
Snippets Groups Projects
Florent Torregrosa's avatar
opened merge request !19 "Resolve #3415791 "Clone display layout builder support"" at project / field_tools
Joshua Sedler's avatar
opened merge request !18 "Issue #3324488: Field groups are not copied when cloning display" at project / field_tools
Joachim Noreiko's avatar
pushed to branch 8.x-1.x at project / field_tools
  • 10540985 · Issue #3469907 by Andrii Momotov: Changed to use short array syntax.
Joachim Noreiko's avatar
pushed to branch 8.x-1.x at project / field_tools
  • 5e8b4adf · Issue #3469912: Replaced t() with $this->t(); removed translation o...
andrii momotov's avatar
opened merge request !17 "Issue #3469915: A lot of unused variables" at project / field_tools
andrii momotov's avatar
opened merge request !16 "Issue #3469913: Add composer.json file" at project / field_tools
andrii momotov's avatar
opened merge request !15 "Issue #3469912: Replace t() with $this->t()" at project / field_tools
andrii momotov's avatar
opened merge request !14 "Issue #3469907: Use short array syntax" at project / field_tools
andrii momotov's avatar
opened merge request !13 "Issue #3469911: Create a new module permission" at project / field_tools
andrii momotov's avatar
opened merge request !12 "Issue #3469906: Set up GitLab CI" at project / field_tools
Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools
}...
Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools

you don't use $key...

Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools

I prefer avoiding nested ifs

Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools

$this->t()

Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools

[]

Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools

dependency injection to be used instead

Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools

use perhaps $this->t() from trait

Ara Martirosyan's avatar
commented on merge request !11 "Issue #3415791: Allow "Clone display" to current bundle view mode (overwrite)" at project / field_tools

I prefer [] for array()

Joachim Noreiko's avatar
Joachim Noreiko's avatar
pushed to branch 8.x-1.x at project / field_tools
  • ed723574 · Added Drupal 11 compatibility.