@hgoto
Upon upgrading to 7.x-2,2. I received this message at the end of update:

Undefined property: OgMembershipType::$language in views_natural_sort_entity_to_vns() (line 405 of /Users/pglatz/project/www/sites/all/modules/contrib/views_natural_sort/views_natural_sort.module).

Perhaps this is related to this OG issue: https://www.drupal.org/node/1783162

CommentFileSizeAuthor
#3 views_natural_sort-fix_undefined_property_issue-2672538-3.patch584 byteshgoto

Comments

pglatz created an issue. See original summary.

hgoto’s picture

hgoto commented
Related issues: +#2677380: Possible conflict with file field or mime mail

Probably this is related with https://www.drupal.org/node/2677380 .

hgoto’s picture

hgoto commented
StatusFileSize
new views_natural_sort-fix_undefined_property_issue-2672538-3.patch584 bytes

Views Natural Sort doesn't check the existence of entity properties when registering the data. But some properties are not always set when Views natural Sort runs registering process. Here is a patch for general solution though I don't know why the $language is not there.

hgoto’s picture

hgoto commented
Status: Active » Needs review

  • generalredneck committed bc15e0c on 7.x-2.x 
    Issue #2672538 by hgoto: Possible conflict with Organic Groups?
generalredneck’s picture

generalredneck
him/his
Primary language English
Location Texas, USA 🇺🇸
commented
Issue summary: View changes
Status: Needs review » Fixed

A fix exactly like yours got in before I go around to this ticket. With that said, I put in a comment adding you contributed and commited it with credit to you.

Lastly, see if the latest dev fixes the problem.

  • generalredneck committed bc15e0c on 8.x-2.x 
    Issue #2672538 by hgoto: Possible conflict with Organic Groups?
hgoto’s picture

hgoto commented

@generalredneck, thank you for your detailed explanation and dealing with this in such a way. Surely, the latest dev fixes the problem. Thank you.

Status: Fixed » Closed (fixed)

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