Calling or hooking local functions of another addon?

[LudiusMaximus]

I have been working for a long time on my fork of the addon DynamicCam, in which I made substantial adjustments regarding the "camera over shoulder offset". I would like to make my changes publicly avaibable to the community, but as my pull request has been ignored so far, all I can do is tell people to manually download my fork code from github. Because surely it would be inappropriate to just upload the entirety of my fork under my name with all the other author's work in it, right?

That's why I thought of offering my changes as a plugin to DynamicCam, but I am facing the obstacle that there are several local functions of DynamicCam's Core.lua file that I would need to call or alter (i.e. hook).

Is there any way at all to do this?

Or what are my other options?

[Tim]

What most people do to sort of bypass the default copyright limitations is to just upload your own version of the addon but, include something along the lines of fan update. (ie: DynamiCam [Fan Update]) However, looking at the GitHub for the addon there is a license file in which it states you can do whatever you want with the code without limitations. Proceed in the direction you feel best at doing so.

[Nimhfree]

Originally Posted by LudiusMaximus <SNIP> That's why I thought of offering my changes as a plugin to DynamicCam, but I am facing the obstacle that there are several local functions of DynamicCam's Core.lua file that I would need to call or alter (i.e. hook). Is there any way at all to do this? Or what are my other options?

I believe there is no way to access the local functions from outside, so the only way really would be to reimplement the local functions with your version and then continue to reimplement the callers all the way up until you get them all.

[Tim]

Originally Posted by Nimhfree I believe there is no way to access the local functions from outside, so the only way really would be to reimplement the local functions with your version and then continue to reimplement the callers all the way up until you get them all.

Correct. When people post code on this forum you will commonly see one of the first things said to them is to convert their unnecessary global functions to their addons local scope to prevent any potential overlapping/interference between other addons. The only way you're going to be able to hook an addons function is if the desired functions are global. You've mentioned you've been using a fork of DynamicCam so really you could just go in and convert the functions to global and then proceed that way.

[Xrystal]

Of course you can always ask the developer to create a global accessor for said function. Whether they will agree is another matter.

[LudiusMaximus]

Originally Posted by Xrystal Of course you can always ask the developer to create a global accessor for said function. Whether they will agree is another matter.

By "global accessor" you mean a global function that can be called instead of the local function, which simply passes its arguments on to a call of the local function?

I doubt that such a request would be successful here as I have not even had a reaction to my pull request for over two months now.

But I shall try the above suggested way of reimplementing the local functions in my own code and override all of DynamicCam's global functions that were calling them.

Thank you all!

[Seerah]

Originally Posted by Tim What most people do to sort of bypass the default copyright limitations is to just upload your own version of the addon but, include something along the lines of fan update. (ie: DynamiCam [Fan Update]) However, looking at the GitHub for the addon there is a license file in which it states you can do whatever you want with the code without limitations. Proceed in the direction you feel best at doing so.

Anyone doing so with an item that is All Rights Reserved is still in violation of Copyright and the addon will be pulled from the site if a complaint is registered or we are aware of such restrictions by the author. (Remember: the absence of a license is the absence of permission, meaning it is All Rights Reserved.)



Now, to the OP...
DynamicCam has this as its license: https://github.com/Mpstark/DynamicCa...ter/LICENSE.md

You are free to modify and redistribute it under its license. I would add "Fan Update" or your name to the file to distinguish it from the original.

[Tim]

Originally Posted by Seerah Anyone doing so with an item that is All Rights Reserved is still in violation of Copyright and the addon will be pulled from the site if a complaint is registered or we are aware of such restrictions by the author. (Remember: the absence of a license is the absence of permission, meaning it is All Rights Reserved.)

Of course I would expect said addon(s) to be removed if there was complaints. However, you can pretty much do a search on the site with the query or "Fan Update" and then compare those to the original addons and they're the same but with perhaps altered/updated code to work with newer versions of the game. I'm not going to go and download all of them and see if there's a license for each to see if they're in violation but, I'm sure more than not do not have a non-all rights reserved license.

[myrroddin]

If no license is specified, then legally it defaults to All Rights Reserved. In that case you must fork and modify your work by a certain percentage. I don't remember what that percentage is, and it probably varies country to country.