I am getting there issues. I am guessing the top one is related to the UUID module. When I disable the UUID module the error goes away.

Notice: Undefined index: vuuid in views_natural_sort_get_supported_entity_properties() (line 333 of /var/www/blog/sites/all/modules/views_natural_sort/views_natural_sort.module).

These two, however, I am at a loss for.

Notice: Undefined property: stdClass::$name in views_natural_sort_entity_to_vns() (line 405 of /var/www/blog/sites/all/modules/views_natural_sort/views_natural_sort.module).
Notice: Undefined property: stdClass::$mime in views_natural_sort_entity_to_vns() (line 405 of /var/www/blog/sites/all/modules/views_natural_sort/views_natural_sort.module).
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

duckydan created an issue. See original summary.

generalredneck’s picture

Ok for the first one...

Notice: Undefined index: vuuid in views_natural_sort_get_supported_entity_properties() (line 333 of /var/www/blog/sites/all/modules/views_natural_sort/views_natural_sort.module).

This turns out to be because uuid is handling revision ids just a hair differently from the node table. I'm going to have to update the code so that it takes into account "revision" tables. when checking for supported properties. In the case of UUID anyway, they are "char" fields and aren't sortable by default anyway... so I can't really test the full impact of the change I'm going to make. I imagine it's going to change with #2664078: Allow Natural Sorting for entity revisions..

For the other, it has to do with a file... I'll have to test to see if it happens on create, edit, or delete... but that's the all you have given me to go on as far as steps to reproduce.

I'll have more details later.

murraybiscuit’s picture

Just updated, same issue here.

jojonaloha’s picture

Status: Active » Needs review
FileSize
1.87 KB

Attached is a patch to fix the php notices.

I was getting this as well.

Notice: Undefined property: stdClass::$name in views_natural_sort_entity_to_vns() (line 405 of /var/www/blog/sites/all/modules/views_natural_sort/views_natural_sort.module).
Notice: Undefined property: stdClass::$mime in views_natural_sort_entity_to_vns() (line 405 of /var/www/blog/sites/all/modules/views_natural_sort/views_natural_sort.module).

It looks like these are from the file entity. I'm using the File Entity module, but I'm not sure if that is required to reproduce or just the File module. Also not sure why the property names are "name" and "mime" but the actual properties on the stdClass object are "filename" and "filemime".

rovo’s picture

I've applied the patch cleanly, but I'm still having issues with UUID .

Notice: Undefined index: vuuid in views_natural_sort_get_supported_entity_properties()

  • generalredneck committed d0d7db0 on 7.x-2.x
    Issue #2669814 by jojonaloha, duckydan: Possible module incompatibility...
generalredneck’s picture

@jojonaloha, @rovo79@gmail.com, @duckydan,

Try this commit that I just posted. There have been a few fixes since the patch that jojonaloha made, so I had to alter the code some. I want to make sure that I covered everything. I did test with uuid enabled and I no longer get the vuuid notice with this fix... but I want to make sure it doesn't break what yall are trying to do.

  • generalredneck committed d0d7db0 on 8.x-2.x
    Issue #2669814 by jojonaloha, duckydan: Possible module incompatibility...
generalredneck’s picture

Status: Needs review » Fixed

Marking as fixed since no input has been added.

Status: Fixed » Closed (fixed)

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