Hello! And sorry for my english!
TVI is a very good module, but one its feature is haunted me - the memory used is up than 150 MB on the pages of adding and editing nodes (node/add*, node/*/edit).
I saw by the Devel module that there are a lot of queries like
tvi_load_settings
SELECT * FROM tvi_settings ts WHERE ts.type = 'vocab' AND ts.xid = my_xid
- they are very small, but there are many such queries (seems to be: number of terms in my site == number of queries).
I thought this topic will help me, but it didn't (possible because of other TVI version there).
Can You help me? I don't deal with code and I don't know are so much queries are needed on such pages.
Great Thank You!
Comments
Comment #1
kevinquillen commentedThat seems odd, because TVI would have no bearing on node add or node edit forms. If this is true this should be rectified immediately.
Comment #2
demoshane commentedI can confirm this. One of your servers that has TVI in use and it's very heavy, see http://monosnap.com/image/pOymry9Vaaw7DOAR2uMw57zKztHatb
Comment #3
duaelfrYou are right. This module have to be entirely rewriten I know it but I unfortunately have other priorities for the moment.
Comment #4
jeffschulerDoes this affect the 7.x version as well?
Comment #5
Leeteq commentedFYI - Ref. #2385633: is this module active again?
Comment #6
kevinquillen commentedTVI is highly useful - but the code does contain legacy cruft from the d6 conversion. A merge or new branch refactor would prove useful.
Comment #7
scott859 commentedI've noticed this too, the memory usage in the D7 version is quite high. I thought about trying to use Panels to override Drupal's default taxonomy path, so I uninstalled TVI and installed Panels and upon enabling the term_view_page in Panels I get the following error:
Page manager module is unable to enable taxonomy/term/%taxonomy_term because some other module already has overridden with views_page.
I have a separate post on this here: https://www.drupal.org/node/2419993
So, it appears I cannot use Panels because TVI did not completely uninstall or at least left the default path taxonomy/term/%taxonomy_term overridden. Maybe it's something else, but this is what it appears to me.
If anyone has any insight on how to solve this I would appreciate it.
Comment #8
scott859 commentedI'm still working on this, but I believe if I disable the view that is overriding the default taxonomy page, that the error I listed above will disappear. At this point that seems to be the best solution if I want to use Panels.
Comment #9
Leeteq commentedComment #10
scott859 commentedIn working with this here is a workaround (to the memory issue) that has helped me:
I am using TVI with Commerce Kickstart, with modified default Drupal taxonomy pages created by a view. Initially, memory usage was very high when applying the view to the taxonomy terms via TVI.
I render the taxonomy term entity in the header of the view. The problem was I needed to have a page view for each term since I couldn't pass an argument (from the URL) to the header. As my number of page views increased, the memory usage was getting quite high.
I found this post where others were having a similar issue and a patch to the Entity API module that allows for arguments to be passed to the header in a view.
Now I can have just one page view and apply it to all my taxonomy term pages. The memory usage has gone way down.
I hope this helps someone with a similar issue.
Comment #11
Leeteq commented@scott859; great, thanks for the tip!
Comment #12
kevinquillen commentedSounds like an issue with a patch for Entity API as mentioned in #10. Closing.