Thread Tools Display Modes
07/16/14, 06:06 AM   #1
esothomas
 
esothomas's Avatar
Join Date: Jun 2014
Posts: 24
"borrowing" code snippets

(didn't know where else to put this, so...)

in reference to this thread, what do we do to emphasize the (potentially) questionable morality displayed?

i know we work in a copy/paste milieu, but that doesn't mean acknowledgements can't be made. it just seems so dishonest to not even mention the source(s).
  Reply With Quote
07/16/14, 07:25 AM   #2
Harven
 
Harven's Avatar
AddOn Author - Click to view addons
Join Date: Apr 2014
Posts: 135
Hey,
I really don't like when people do copy/paste work without even stating that they used someones else code, but I think in this case accusations are exaggerated. I see only one copy/paste function from LightWeightMinimap (16 lines of code). The addon structure (folders and file names) are similar to RMM and so are few function names. You can say that the structure is inspired by RMM. But generally the code is very different from LightWeightMinimap and RMM. I don't think it deserves to be blocked or something because this isn't staight copy/paste/mix work. I agree that the author should write in the description that his addon was inspired by RMM and LightWeightMinimap but again this accusation: "The author has copied LightWeightMinimap and RMM in a new addon." is really exaggerated and unjust.
My opinion is based on about 30 minutes of looking at these three addons so maybe more deep review will reveal more copy/paste work but I think will be no more than few percent of the whole code.
  Reply With Quote
07/16/14, 08:36 AM   #3
zgrssd
AddOn Author - Click to view addons
Join Date: May 2014
Posts: 280
If you think it really was stolen, just report it. The team will take a closer look at it. And if they disagree nothing will happen.

It is hard to know where exactly the border between insipiration, copy&paste and "great minds think alike" lies.
And it is quite possible that the functions look simialr is only conincidence:
There is often only one way to solve a given problem. That two people who solved the same problem happened to have the same solution is nothing odd.

I don't need to look at the code to guess that UnifiedChatTabs and ChatSetup propably have a few pieces of code in common. They work on the same thing (reading and writing of chat tab data & category colors). They solve the same issues. So naturally they might do so in the same way.
If not I propably look wich solution is better (mine or Xerovan) and just adapt it.
  Reply With Quote
07/16/14, 09:56 AM   #4
esothomas
 
esothomas's Avatar
Join Date: Jun 2014
Posts: 24
yeah, i just wanted to run it by someone before i got all up in arms and stuff.

of course, i'd like for creators to be honest enough to state their "borrowing" clearly, but such is not always the case...

is there even a "report" function on this site?
  Reply With Quote
07/16/14, 01:22 PM   #5
zgrssd
AddOn Author - Click to view addons
Join Date: May 2014
Posts: 280
Originally Posted by esothomas View Post
yeah, i just wanted to run it by someone before i got all up in arms and stuff.

of course, i'd like for creators to be honest enough to state their "borrowing" clearly, but such is not always the case...

is there even a "report" function on this site?
At the bottom of the Addon Page.
Just above the Category Jump System.
It is called "report this File".
  Reply With Quote
07/16/14, 04:39 PM   #6
esothomas
 
esothomas's Avatar
Join Date: Jun 2014
Posts: 24
Originally Posted by zgrssd View Post
At the bottom of the Addon Page.
yeah i guess one of those categories would do. ah well... monitoring the situation for now, methinks.
  Reply With Quote
07/26/14, 06:52 PM   #7
deathangel1479
 
deathangel1479's Avatar
AddOn Author - Click to view addons
Join Date: May 2014
Posts: 6
Goes on with zrmm...
Or I am paranoid?
I give two example...

1.
Abstract function, created by me. Its a bad result, but it works great for cyrodil links update...
That is one thing with a impossible chance, that two author make same...
RMM:
Lua Code:
  1. --Show Map Hidden
  2. function RMM.WorldMapUpdate()
  3.     ZO_WorldMap_UpdateMap()
  4.     ZO_WorldMap:SetAlpha(0)
  5.     ZO_WorldMap:SetHidden(false)
  6.     zo_callLater(RMM.WorldMapReset, 100)
  7. end
  8.  
  9. --Hide Map
  10. function RMM.WorldMapReset()
  11.     ZO_WorldMap:SetHidden(true)
  12.     ZO_WorldMap:SetAlpha(1)
  13. end
zrmm:
Lua Code:
  1. --------------------------------------------------
  2. -- WorlMap refresh
  3. --------------------------------------------------
  4. function MM_WorldMapRefresh()
  5.     ZO_WorldMap:SetAlpha(0)
  6.     ZO_WorldMap:SetHidden(false)
  7.     ZO_WorldMap_PanToPlayer()
  8.     zo_callLater(MM_WorldMapRefreshStop,20)
  9. end
  10.  
  11. function MM_WorldMapRefreshStop()
  12.     ZO_WorldMap:SetHidden(true)
  13.     ZO_WorldMap:SetAlpha(1)
  14. end
OK, its not the same, he changed a line and changed callLater value... :P

2.
zrmm now has tooltips, I added it two weeks ago(two days work to understand correctly, rebuild from worldmap and much testing).

True, some thinks you must do like worldmap, so there must be parallels, but so exactly same?

The worldmap build a list of tooltips when you hover more than one pins.
I dont do it, because to much work, single was ok for me...
WOW, what a coincidence, same for zrmm author...



What he thinking?
That I code new features for him?
"I can concentrate on stable and performans and all other I get from other addons." Or what?

I got the result for nearly no fps drop(much better as worldmap itself), but I wont build it, because it is real much work and I am sure, other will be copy it then...

Last edited by deathangel1479 : 07/26/14 at 07:05 PM.
  Reply With Quote
07/27/14, 10:47 PM   #8
Sasky
AddOn Author - Click to view addons
Join Date: Apr 2014
Posts: 231
Yes, it does look very similar. However, this isn't something that should be aired in the forums unless the ESOUI admins are completely unresponsive.

1. Contact the author directly for him to remove the code or add credit.
2. If that doesn't go well, contact one of the admins - PM or using the Report File option. Note sections to make your point and indicate any results from (1).
3. If you don't get a response or appropriate action from the admins, then consider airing in the forums.

#1 and #2 can get something done about it. #3 cannot, unless the author or an admin review the thread, in which case #1 or #2 would've gotten you directly there.
  Reply With Quote
07/27/14, 11:23 PM   #9
Cairenn
Credendo Vides
 
Cairenn's Avatar
Premium Member
WoWInterface Admin
Join Date: Mar 2004
Posts: 437
Originally Posted by Sasky View Post
Yes, it does look very similar. However, this isn't something that should be aired in the forums unless the ESOUI admins are completely unresponsive.

1. Contact the author directly for him to remove the code or add credit.
2. If that doesn't go well, contact one of the admins - PM or using the Report File option. Note sections to make your point and indicate any results from (1).
3. If you don't get a response or appropriate action from the admins, then consider airing in the forums.

#1 and #2 can get something done about it. #3 cannot, unless the author or an admin review the thread, in which case #1 or #2 would've gotten you directly there.
This. A thousand times, this.
  Reply With Quote

ESOUI » Developer Discussions » General Authoring Discussion » "borrowing" code snippets


Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off