Ok, to be honest, I did not really try the module yet :)
I just know that unnecessary views loading is bad for performance.

eva_get_views() does use the cache, this gives hope.
Still, it can happen that eva_entity_view_alter() does load and execute a few views, when the result might be discarded later on.
I don't see any switch that would disable this function altogether.

I personally would like to use eva views only as ds/nd fields, and nothing else. Especially, I don't want to spend any execution time for any views display that is not configured as a field with display suite.
I even made a D6 replacement of views_attach with the same purpose ..

Any idea how we could skip that hook_entity_view_alter() ?
With ds/nd enabled, we probably don't need it at all, do we?

------------

Btw, about views_get_applicable_views(), I hope the future can give us something better..
#1378372: views_get_applicable_views() - performance.

Comments

donquixote’s picture

donquixote commented

Just to clarify my motivation,
I've "been there, done that" in D6, so now I am a bit paranoid if eva will do it any better. Maybe it does :)

mkadin’s picture

mkadin commented
Status: Active » Closed (won't fix)

At this point, unless someone else wants to step in, I don't think we're going to reorganize the way EVA attaches its content. For better or for worse.