Pass target object when evaluating expression for Generic Object

[yrudman]

*Issue:
call toGenericObject#custom_actions was delegated to GenericObjectDefinition. As a result, evaluating expression defined on GenericObject would get GenericObjectDefinition as target (instead of GenericObject)
Resolves https://bugzilla.redhat.com/show_bug.cgi?id=1553767

There was similar issue with evaluating expression for Service addressed here: #16770
BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1505988

\cc @gtanzillo @gmcculloug

[miq-bot]

Checked commits yrudman/manageiq@e654e16~...4b1d05d with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0
3 files checked, 0 offenses detected
Everything looks fine. 🏆

[yrudman]

@miq-bot add-label gaprindashvili/yes, bug, core

[simaishi]

@yrudman Since the referenced BZ is for service (and already closed), need a different BZ to track this particular change.

[yrudman]

@simaishi that BZ (https://bugzilla.redhat.com/show_bug.cgi?id=1505988) is in POST, can i just add another PR to it ?

[simaishi]

@yrudman You'll need to look at the 5.9 clone since this is a backport to gaprindashvili branch
https://bugzilla.redhat.com/show_bug.cgi?id=1535153

We can't use the same BZ when 1 PR is already in a released downstream build and BZ is CLOSED/ERRATA.

And since that BZ is specifically talking about Service, we should track this separately anyway...

[yrudman]

@simaishi BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1553767

[simaishi]

Gaprindashvili backport details:

$ git log -1
commit 87c54f142f371811366eea13d351b4bdd0d91b1b
Author: Oleg Barenboim <[email protected]>
Date:   Tue Jan 23 11:48:32 2018 -0500

    Merge pull request #16858 from yrudman/fixed-expression-evaluation-for-generic-object
    
    Pass target object when evaluating expression for Generic Object
    (cherry picked from commit 20c5a8446f35935d9bf28afb5bf8c3b237a76077)
    
    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1553836