CommentFileSizeAuthor
#187 1848068-187.patch26.09 KBaspilicious
#184 1848068-185.patch26.11 KBswentel
#183 1848068-183.patch26.6 KBswentel
#181 1848068-181.patch28.02 KBswentel
#179 1875974-entity-components-andypost-65.patch33.05 KBandypost
#179 interdiff.txt1.58 KBandypost
#176 1875974-entity-components-andypost-65.patch32.37 KBandypost
#176 interdiff.txt2.72 KBandypost
#175 1875974-entity-components-andypost-65.patch32.76 KBandypost
#175 interdiff.txt899 bytesandypost
#173 1875974-entity-components-andypost-65.patch32.59 KBandypost
#173 interdiff.txt5.97 KBandypost
#171 1875974-entity-components-andypost-65.patch30.97 KBandypost
#170 1875974-entity-components-60.patch31.25 KBandypost
#170 interdiff.txt8.79 KBandypost
#168 test.patch124.43 KBandypost
#165 test.patch121.22 KBandypost
#165 interdiff.txt1.85 KBandypost
#162 test.patch121.07 KBandypost
#162 interdiff.txt7.1 KBandypost
#160 test-di.patch117.75 KBandypost
#160 interdiff-di.txt9.1 KBandypost
#158 test.patch115.06 KBandypost
#158 interdiff.txt1.27 KBandypost
#156 1848068-156.patch114.97 KBaspilicious
#153 1848068-154.patch124.32 KBaspilicious
#152 1848068-153.patch124.38 KBaspilicious
#151 1848068-151.patch125.88 KBaspilicious
#148 1848068-148.patch149.97 KBaspilicious
#147 1848068-147.patch149.99 KBaspilicious
#145 1848068-145.patch162.11 KBaspilicious
#144 1848068-144.patch161.23 KBaspilicious
#142 1848068-141.patch161.2 KBaspilicious
#141 1848068-139.patch161.18 KBaspilicious
#139 1848068-137.patch160.68 KBaspilicious
#137 1848068-135.patch157.76 KBaspilicious
#134 1848068-132.patch151.6 KBaspilicious
#131 1848068-131.patch166.47 KBaspilicious
#128 1848068-128.patch166.55 KBaspilicious
#126 drupal8.entity-system.1848068-126.patch155.87 KBandypost
#124 interdiff.txt11.22 KBandypost
#124 drupal8.entity-system.1848068-124.patch155.87 KBandypost
#122 interdiff.txt6.02 KBamateescu
#120 1848068-120.patch160.04 KBswentel
#118 1848068-118.patch159.69 KBswentel
#116 1848068-116.patch160.08 KBswentel
#114 1848068-114.patch161.97 KBswentel
#110 1848068-110.patch161.85 KBaspilicious
#108 1848068-108.patch159.06 KBswentel
#106 1848068-106.patch158.29 KBswentel
#104 1848068-104.patch157.17 KBamateescu
#102 1848068-102.patch117.29 KBamateescu
#101 1848068-101.patch117.63 KBamateescu
#99 1848068-99.patch119.3 KBswentel
#97 1848068-97.patch115.16 KBswentel
#95 1848068-94.patch121.33 KBswentel
#93 1848068-93.patch119.21 KBswentel
#89 1848068-89.patch123.35 KBswentel
#88 1848068-88.patch122.01 KBswentel
#88 interdiff.txt3.9 KBswentel
#86 1848068-86.patch120.43 KBswentel
#86 interdiff.txt2.84 KBswentel
#84 1848068-84.patch120.41 KBswentel
#82 1848068-82.patch112.34 KBswentel
#80 1848068-80.patch112.95 KBswentel
#77 display-handlers-testbot-1848068-77.patch112.43 KBswentel
#71 display-handlers-testbot-1848068-71.patch112.38 KBswentel
#71 interdiff.txt1.24 KBswentel
#69 display-handlers-testbot-1848068-69.patch112.01 KBswentel
#69 interdiff.txt2.62 KBswentel
#67 display-handlers-testbot-1848068-67.patch109.4 KBswentel
#67 interdiff.txt9.27 KBswentel
#65 display-handlers-testbot-1848068-65.patch102.78 KByched
#63 display-handlers-testbot-1848068-63.patch101.22 KByched
#62 display-hooks-testbot-1848068-62.patch54.22 KByched
#60 entity_display-testbot-1848068-60.patch214.52 KByched
#58 entity_display-testbot-1848068-58.patch209.63 KByched
#56 entity_display-testbot-1848068-56.patch205.42 KByched
#54 entity_display-testbot-1848068-54.patch205.42 KByched
#52 entity_display-testbot-1848068-52.patch197 KByched
#51 entity_display-testbot-1848068-51.patch198.66 KByched
#49 entity_display-testbot-1848068-49.patch198.66 KByched
#47 entity_display-testbot-1848068-47.patch198.28 KByched
#45 entity_display-testbot-1848068-44.patch191.57 KByched
#43 entity_display-testbot-1848068-42.patch172.39 KByched
#42 entity_display-testbot-1848068-42.patch172.39 KByched
#40 entity_display-testbot-1848068-40.patch171.4 KByched
#36 entity_display-testbot-1848068-35.patch171.4 KByched
#35 entity_display-testbot-1848068-34.patch170.88 KByched
#34 entity_display-testbot-1848068-34.patch170.87 KByched
#33 entity_display-testbot-1848068-33.patch174.47 KByched
#31 entity_display-testbot-1848068-31.patch162.62 KByched
#30 entity_display-testbot-1848068-30.patch155.08 KByched
#28 entity_display-testbot-1848068-28.patch150.79 KByched
#26 entity_display.patch141.1 KByched
#24 entityDisplay-testbot-1848068-24.patch138.33 KByched
#22 entityDisplay-testbot-1848068-23.patch135.39 KByched
#20 entityDisplay-testbot-alter-1848068-21.patch136.47 KByched
#18 entityDisplay-testbot-streamlined-1848068-19.patch129.95 KByched
#16 entityDisplay-testbot-streamlined-1848068-17.patch129.99 KByched
#14 entityDisplay-testbot-streamlined-1848068-14.patch129.52 KByched
#12 entityDisplay-testbot-formatter_in_display-1848068-12.patch131.54 KByched
#10 entityDisplay-testbot-formatter_in_display-1848068-10.patch131.6 KByched
#8 entityDisplay-testbot-formatter_in_display-1848068-6.patch131.69 KByched
#4 1830868-32-EntityDisplay_0.patch132.85 KByched
#3 entityDisplay-testbot-formatter_in_display-1848068-3.patch150.26 KByched
#2 entityDisplay-official-branch-1848068-2.patch121.23 KByched
#1 entityDisplay-official-branch-1848068-1.patch120.79 KByched

Comments

yched’s picture

Status: Active » Needs review
StatusFileSize
new120.79 KB
yched’s picture

StatusFileSize
new121.23 KB
yched’s picture

yched’s picture

StatusFileSize
new132.85 KB

Status: Needs review » Needs work

The last submitted patch, 1830868-32-EntityDisplay_0.patch, failed testing.

yched’s picture

Status: Needs work » Needs review

#4: 1830868-32-EntityDisplay_0.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1830868-32-EntityDisplay_0.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new131.69 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-formatter_in_display-1848068-6.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new131.6 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-formatter_in_display-1848068-10.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new131.54 KB

Stupid me.

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-formatter_in_display-1848068-12.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new129.52 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-streamlined-1848068-14.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new129.99 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-streamlined-1848068-17.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new129.95 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-streamlined-1848068-19.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new136.47 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-alter-1848068-21.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new135.39 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-1848068-23.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new138.33 KB

Status: Needs review » Needs work

The last submitted patch, entityDisplay-testbot-1848068-24.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new141.1 KB

Status: Needs review » Needs work

The last submitted patch, entity_display.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new150.79 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-28.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new155.08 KB
yched’s picture

StatusFileSize
new162.62 KB

method renames

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-31.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new174.47 KB
yched’s picture

StatusFileSize
new170.87 KB
yched’s picture

StatusFileSize
new170.88 KB
yched’s picture

StatusFileSize
new171.4 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-35.patch, failed testing.

yched’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-35.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new171.4 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-40.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new172.39 KB
yched’s picture

StatusFileSize
new172.39 KB

Stuck ?

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-42.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new191.57 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-44.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new198.28 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-47.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new198.66 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-49.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new198.66 KB
yched’s picture

StatusFileSize
new197 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-52.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new205.42 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-54.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new205.42 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-56.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new209.63 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-58.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new214.52 KB

Status: Needs review » Needs work

The last submitted patch, entity_display-testbot-1848068-60.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new54.22 KB
yched’s picture

StatusFileSize
new101.22 KB

Status: Needs review » Needs work

The last submitted patch, display-handlers-testbot-1848068-63.patch, failed testing.

yched’s picture

Status: Needs work » Needs review
StatusFileSize
new102.78 KB

Status: Needs review » Needs work

The last submitted patch, display-handlers-testbot-1848068-65.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new9.27 KB
new109.4 KB

Kickstarting again, we really really need this for field group, ds and let contrib go nuts! :)

Status: Needs review » Needs work

The last submitted patch, display-handlers-testbot-1848068-67.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new2.62 KB
new112.01 KB

Should fix the remaining tests.

Status: Needs review » Needs work

The last submitted patch, display-handlers-testbot-1848068-69.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new1.24 KB
new112.38 KB

This should be green

Status: Needs review » Needs work

The last submitted patch, display-handlers-testbot-1848068-71.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
swentel’s picture

Stalski’s picture

Hi, I am looking for the @todo interface in the patch.
Just a question, could it be there is a left over in the patch: there are two abstract class DisplayComponentHandlerBase files. I think the "Drupal\entity" namespaced one is not used anymore?

yched’s picture

Ew - right, Drupal\entity\DisplayComponentHandlerBase.php is stale.
For the interface, I was just waiting to settle on the actual methods in the handlers.

swentel’s picture

StatusFileSize
new112.43 KB

Just keeping up with HEAD

Status: Needs review » Needs work

The last submitted patch, display-handlers-testbot-1848068-77.patch, failed testing.

swentel’s picture

Oh right, custom blocks and datetime field have landed too

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new112.95 KB

reviving, branch opened in sandbox : entity-display-components-1875974 (I kind of lost the old one apparently)

Had to merge a lot, got notices on standard install, want to give this a roll though.

Status: Needs review » Needs work

The last submitted patch, 1848068-80.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new112.34 KB

Fix discovery

Status: Needs review » Needs work

The last submitted patch, 1848068-82.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new120.41 KB

More obvious fixes

Status: Needs review » Needs work

The last submitted patch, 1848068-84.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new2.84 KB
new120.43 KB

Seriously

Status: Needs review » Needs work

The last submitted patch, 1848068-86.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new3.9 KB
new122.01 KB
swentel’s picture

StatusFileSize
new123.35 KB

Messy merge with HEAD, let's see what the bot thinks.

Status: Needs review » Needs work

The last submitted patch, 1848068-89.patch, failed testing.

effulgentsia’s picture

Status: Needs work » Needs review

#89: 1848068-89.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 1848068-89.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new119.21 KB

re-kickstarting (for the 4th time) on top of the new form modes patch, let's see how this works out.

- also - I switched branches - 1875974-entity-components

Status: Needs review » Needs work

The last submitted patch, 1848068-93.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new121.33 KB

This should have a lot more fixes - a little too much code duplication, need to clean that up at some point.

Status: Needs review » Needs work

The last submitted patch, 1848068-94.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new115.16 KB

Use 'type' again instead of formatter

Status: Needs review » Needs work

The last submitted patch, 1848068-97.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new119.3 KB

Fix more tests, amateescu made the pluginmanager to modern times

Status: Needs review » Needs work

The last submitted patch, 1848068-99.patch, failed testing.

amateescu’s picture

Status: Needs work » Needs review
StatusFileSize
new117.63 KB

Reverted some stuff and fixed the default config files.

amateescu’s picture

StatusFileSize
new117.29 KB

Renamed the 'widget' handler type back to 'field'.

Status: Needs review » Needs work

The last submitted patch, 1848068-102.patch, failed testing.

amateescu’s picture

Status: Needs work » Needs review
StatusFileSize
new157.17 KB

Fixed all getComponent() and setComponent() calls.

Status: Needs review » Needs work

The last submitted patch, 1848068-104.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new158.29 KB

More fixes - should fix a lot of notices in the upgrade path

Status: Needs review » Needs work

The last submitted patch, 1848068-106.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new159.06 KB

Even more fixes

Status: Needs review » Needs work

The last submitted patch, 1848068-108.patch, failed testing.

aspilicious’s picture

Status: Needs work » Needs review
StatusFileSize
new161.85 KB

Helping this forward, fixing some tests.

swentel’s picture

aspilicious, can you commit those fixes to the sandbox also ? 1875974-entity-components

Status: Needs review » Needs work

The last submitted patch, 1848068-110.patch, failed testing.

aspilicious’s picture

I'm a good boy did that *before* posting this patch :D

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new161.97 KB

New patch against latest HEAD

Status: Needs review » Needs work

The last submitted patch, 1848068-114.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new160.08 KB

More fixes, getting close to green (again).

Status: Needs review » Needs work

The last submitted patch, 1848068-116.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new159.69 KB

Status: Needs review » Needs work

The last submitted patch, 1848068-118.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new160.04 KB

Fix tons of notices

Status: Needs review » Needs work

The last submitted patch, 1848068-120.patch, failed testing.

Anonymous’s picture

Issue summary: View changes

update issue #

amateescu’s picture

StatusFileSize
new6.02 KB

Started to work on this but I got very confused when I got to DisplayComponentHandlerBase::getRenderer() because we also have a getRenderer() method in EntityDisplayBase. Was the original intention to move it from the config entity class to the new plugin type (component handler)?

Posting just an interdiff for now, nothing committed anywhere :/

yched’s picture

I think the intention was:
- external world calls $display-getRenderer($field_name)
- internally the display does $this->handler($field_name)->getRendeter() (and keeps a static cache of the results)

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new155.87 KB
new11.22 KB

Suppose $type should be optional for getComponent(), we already store handler_type in display so 123 could work
But for setComponent() we need to know the the type of component!

Merged current HEAD and made some clean-up, all pushed to 1875974-entity-components-andypost

Status: Needs review » Needs work

The last submitted patch, drupal8.entity-system.1848068-124.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new155.87 KB

..drop is moving

Status: Needs review » Needs work

The last submitted patch, drupal8.entity-system.1848068-126.patch, failed testing.

Anonymous’s picture

Issue summary: View changes

Updated issue summary.

aspilicious’s picture

Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new166.55 KB

I could install and some of the field stuff worked, lets see

aspilicious’s picture

Status: Needs review » Needs work

The last submitted patch, 128: 1848068-128.patch, failed testing.

aspilicious’s picture

StatusFileSize
new166.47 KB
aspilicious’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 131: 1848068-131.patch, failed testing.

aspilicious’s picture

StatusFileSize
new151.6 KB

Now without upgrade path tests

aspilicious’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, 134: 1848068-132.patch, failed testing.

aspilicious’s picture

Status: Needs work » Needs review
StatusFileSize
new157.76 KB

Should fix most problems, I hope

Status: Needs review » Needs work

The last submitted patch, 137: 1848068-135.patch, failed testing.

aspilicious’s picture

Status: Needs work » Needs review
StatusFileSize
new160.68 KB

More fixes! Less notices!

Status: Needs review » Needs work

The last submitted patch, 139: 1848068-137.patch, failed testing.

aspilicious’s picture

Status: Needs work » Needs review
StatusFileSize
new161.18 KB

Lets try this one

aspilicious’s picture

StatusFileSize
new161.2 KB

Less notices on this one.

The last submitted patch, 141: 1848068-139.patch, failed testing.

aspilicious’s picture

StatusFileSize
new161.23 KB
aspilicious’s picture

StatusFileSize
new162.11 KB

The last submitted patch, 142: 1848068-141.patch, failed testing.

aspilicious’s picture

StatusFileSize
new149.99 KB
aspilicious’s picture

StatusFileSize
new149.97 KB

The last submitted patch, 144: 1848068-144.patch, failed testing.

The last submitted patch, 144: 1848068-144.patch, failed testing.

aspilicious’s picture

StatusFileSize
new125.88 KB
aspilicious’s picture

StatusFileSize
new124.38 KB

Fixed some minor issues

aspilicious’s picture

StatusFileSize
new124.32 KB

Another reroll

The last submitted patch, 147: 1848068-147.patch, failed testing.

The last submitted patch, 148: 1848068-148.patch, failed testing.

aspilicious’s picture

StatusFileSize
new114.97 KB

Status: Needs review » Needs work

The last submitted patch, 156: 1848068-156.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new1.27 KB
new115.06 KB

No configuration is prepared so a lot of tests fail

Status: Needs review » Needs work

The last submitted patch, 158: test.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
Parent issue: » #1875974: Abstract 'component type' specific code out of EntityDisplay
StatusFileSize
new9.1 KB
new117.75 KB

Another set of changes:
- fix for massageIn() that executed for non-existing field - needs some research
- field handler now injects needed services
- some doc blcok fixes
- added todos

Status: Needs review » Needs work

The last submitted patch, 160: test-di.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new7.1 KB
new121.07 KB

Renamed massageOut() => onComponentGet()
Decoupled massageIn() => onComponentSet() and onComponentRemove()
Added doc-blocks

Status: Needs review » Needs work

The last submitted patch, 162: test.patch, failed testing.

aspilicious’s picture

In the onComponentSet function you need to "change" the options and return the new options.
You're returning a new array overriding the past options.

==> test fails :)

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new1.85 KB
new121.22 KB

should fix failed tests, Field handler needs better check for removed field

yched’s picture

No time to look into the interdiffs right now, but +1 on the onXxx() method renames, I was thinking the same while brushing my teeth yesterday night :-)

As for "Handler plugins persisted and shared in the Manager / setContext()" :
Yeah, that was an attempt to avoid creating 2 (or more) handler plugin objects for each EntityDisplay (that is, for each bundle in an entity_view() / entity_view_multiple()). Instead, patch creates one instance of each "handler type" and reuses them for the whole request, injecting the proper context each time with setContext().

I'm not really happy with that either. I fully agree that it is not exactly intuitive, and also probably doesn't play really well with nested entity_view() calls (e.g an entity_ref formatter displaying the referenced entity). Might be a premature optimisation, but hard to tell the exact impact without actual benchmarks though - and entity_view() is definitely a very critical path...

The best route would probably be to remove it for now and switch to a more common pattern of "each EntityDisplay has a PluginBag of Handler plugins". Then benchmark and see if/how we can optimize from there.

Status: Needs review » Needs work

The last submitted patch, 165: test.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new124.43 KB

Status: Needs review » Needs work

The last submitted patch, 168: test.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new8.79 KB
new31.25 KB

Let's test with interface

andypost’s picture

StatusFileSize
new30.97 KB

check merge

Status: Needs review » Needs work

The last submitted patch, 171: 1875974-entity-components-andypost-65.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new5.97 KB
new32.59 KB

Status: Needs review » Needs work

The last submitted patch, 173: 1875974-entity-components-andypost-65.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new899 bytes
new32.76 KB

One more

andypost’s picture

StatusFileSize
new2.72 KB
new32.37 KB

Use core's pattern to get field definitions

The last submitted patch, 175: 1875974-entity-components-andypost-65.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 176: 1875974-entity-components-andypost-65.patch, failed testing.

andypost’s picture

Status: Needs work » Needs review
StatusFileSize
new1.58 KB
new33.05 KB

render pipeline should change, this just a hack that not commited

Status: Needs review » Needs work

The last submitted patch, 179: 1875974-entity-components-andypost-65.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new28.02 KB

Status: Needs review » Needs work

The last submitted patch, 181: 1848068-181.patch, failed testing.

swentel’s picture

Status: Needs work » Needs review
StatusFileSize
new26.6 KB
swentel’s picture

StatusFileSize
new26.11 KB

The last submitted patch, 183: 1848068-183.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 184: 1848068-185.patch, failed testing.

aspilicious’s picture

Status: Needs work » Needs review
StatusFileSize
new26.09 KB

Lets see...

Version: 8.0.x-dev » 8.1.x-dev

Drupal 8.0.6 was released on April 6 and is the final bugfix release for the Drupal 8.0.x series. Drupal 8.0.x will not receive any further development aside from security fixes. Drupal 8.1.0-rc1 is now available and sites should prepare to update to 8.1.0.

Bug reports should be targeted against the 8.1.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.2.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.1.x-dev » 8.2.x-dev

Drupal 8.1.9 was released on September 7 and is the final bugfix release for the Drupal 8.1.x series. Drupal 8.1.x will not receive any further development aside from security fixes. Drupal 8.2.0-rc1 is now available and sites should prepare to upgrade to 8.2.0.

Bug reports should be targeted against the 8.2.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.3.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.2.x-dev » 8.3.x-dev

Drupal 8.2.6 was released on February 1, 2017 and is the final full bugfix release for the Drupal 8.2.x series. Drupal 8.2.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.3.0 on April 5, 2017. (Drupal 8.3.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.3.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.4.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.3.x-dev » 8.4.x-dev

Drupal 8.3.6 was released on August 2, 2017 and is the final full bugfix release for the Drupal 8.3.x series. Drupal 8.3.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.4.0 on October 4, 2017. (Drupal 8.4.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.4.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.5.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.4.x-dev » 8.5.x-dev

Drupal 8.4.4 was released on January 3, 2018 and is the final full bugfix release for the Drupal 8.4.x series. Drupal 8.4.x will not receive any further development aside from critical and security fixes. Sites should prepare to update to 8.5.0 on March 7, 2018. (Drupal 8.5.0-alpha1 is available for testing.)

Bug reports should be targeted against the 8.5.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.6.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

Version: 8.5.x-dev » 8.6.x-dev

Drupal 8.5.6 was released on August 1, 2018 and is the final bugfix release for the Drupal 8.5.x series. Drupal 8.5.x will not receive any further development aside from security fixes. Sites should prepare to update to 8.6.0 on September 5, 2018. (Drupal 8.6.0-rc1 is available for testing.)

Bug reports should be targeted against the 8.6.x-dev branch from now on, and new development or disruptive changes should be targeted against the 8.7.x-dev branch. For more information see the Drupal 8 minor version schedule and the Allowed changes during the Drupal 8 release cycle.

amateescu’s picture

Status: Needs review » Closed (outdated)

I think further work on this can happen in the main issue now :)