<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 11, 2013 at 10:22 AM, Christian Loos <span dir="ltr"><<a href="mailto:cloos@netcologne.de" target="_blank">cloos@netcologne.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Am 10.06.2013 20:36, schrieb Ruslan Zakirov:<br>
<div class="im">> see lib/RT/IR.pm in RTIR extension. We basicly wrap methods from<br>
> extensions' primary file.<br>
<br>
</div>You use here Hook::LexWrap and I saw some commit comments that<br>
Hook::LexWrap makes some troubles:<br>
<a href="https://github.com/bestpractical/rt-extension-mergeusers/commit/354edf0" target="_blank">https://github.com/bestpractical/rt-extension-mergeusers/commit/354edf0</a></blockquote><div><br></div><div style>In recent code we don't use Hook::LexWrap as it's in most case unnecesary. Something close to the following should work just fine:</div>
<div style><br></div><div style>require RT::XXX;</div><div style>{</div><div style>    my $orig = RT::XXX->can("method");</div><div style>    *RT::XXX::method = sub {</div><div style>        my $self = shift;</div>
<div style>        return $orig->($self, @_);</div><div style>    };</div><div style>}</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

Chris<br>
<div class=""><div class="h5"><br>
<br>
--<br>
RT Training in Seattle, June 19-20: <a href="http://bestpractical.com/training" target="_blank">http://bestpractical.com/training</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Best regards, Ruslan.
</div></div>