I've been getting these errors when I view my user account details : http://example.com/user or any other user's account.

  • Notice: Undefined index: #entity_type in _eva_extract_entity_from_build() (line 190 of /var/www/sites/all/modules/eva/eva.module).
  • Notice: Undefined index: #entity_type in _eva_extract_entity_from_build() (line 198 of /var/www/sites/all/modules/eva/eva.module).
  • Notice: Undefined index: #entity_type in _eva_extract_entity_from_build() (line 201 of /var/www/sites/all/modules/eva/eva.module).

I have resolved this issue with the forthcoming patch.

CommentFileSizeAuthor
#6 interdiff-2414649-1-6.txt810 bytesheykarthikwithu
#6 2414649-6.patch561 bytesheykarthikwithu
#1 build_entity_type_doesnt_always_get_set-2414649-1.patch906 bytesdarrellduane

Comments

darrellduane’s picture

darrellduane commented
StatusFileSize
new build_entity_type_doesnt_always_get_set-2414649-1.patch906 bytes

Here is a patch that suppresses these error messages.

morsok’s picture

morsok
Primary language French
Location Paris
commented

Patch worked and solved the notice for me too, if i may be picky on the patch regarding coding standards :

+++ b/eva.module
@@ -181,6 +181,11 @@ function eva_get_views($type = NULL, $reset = FALSE) {
+  if(empty($build['#entity_type'])) {                                                                                                                                                                          ¶

Needs a space between if and the parenthesis.

morsok’s picture

morsok
Primary language French
Location Paris
commented
Status: Active » Needs review
geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Status: Needs review » Needs work

This looks good code-wise, but see above and patch has trailing whitespace.
Pleease re-roll as an attached patch file.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Issue tags: +Novice

This looks good code-wise, but see above and patch has trailing whitespace.
Pleease re-roll as an attached patch file.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Status: Needs work » Needs review
StatusFileSize
new 2414649-6.patch561 bytes
new interdiff-2414649-1-6.txt810 bytes

re-roll patch.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented

Codewise perfect. If s.o. applies and confirms everything works and it does not eat our hamster (although i can't imagine how it should) this is RTBC.

heykarthikwithu’s picture

heykarthikwithu
Primary language English
Location Bengaluru 🌍
commented
Status: Needs review » Reviewed & tested by the community

  • axel.rutz committed f619c1c on 7.x-1.x 
    Issue #2414649 by heykarthikwithu, DarrellDuane, axel.rutz: Undefined...
geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented

Fixed in 1.4-rc1, which will become 1.4 if enough people use it without bugs reported. See #2647732: Plan for EVA 7.x-1.4 release

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed

This was committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.