Make _votingapi_delete_votes_by_entity receive uid of entity

Created a patch adding entity uid.

Added db selection of vote and tag fields. They are also needed to calculate userpoints properly.

Bump.

Thank you for your answer.
I think, votingapi means that only entities with uid can be voted (nodes and comments), doesn't it? So it would be safe to get that property.
And that makes sence, because the usecase is actually found. Though I can't imagine the usecase where the whole entity should be passed, but let it be.

So if I post the patch that passes the whole entity, will you review it for accepting?

I'll go though my corrections.)

1. - $id = array($ids[0]);
We don't use an array of ids. The function returns only one entity_id with its vid and bundle, so there is no sense to be it as an array. Also in the functions to delete cache and votes you use "->condition('entity_id', $entity_ids)" which wasn't correct (we should use 'IN' operator then).

2. _votingapi_delete_votes_by_entity($entity_id, $type, $entity = NULL)
It may seem that we have excess $entity_id, but I decieded to retain it in order to save compability with function "_votingapi_cron_delete_orphaned", so passing $entity is still optional. I couldn't get calling

DrupalQueue::get('VotingAPIOrphaned') 

to return any value of testing purposes, so I left it alone.

Hope that's all okay so you'll accept my patch.)
Thank you :)

Bump.

Please, review my last changes and commit them if possible.
My further development of userpoints_votingapi is frozen until this patch is applied.