AddonLoader

16 - LDB launcher and text field

What steps will reproduce the problem?
1. Create a mod with an LDB feed with a text field and use addonloader recommended procedure to make it LOD
2. When you use ldb:GetDataObjectByName() to get the DO made by addonloader it has no text field
3. Therefore when you try to access the text field to display your text you will usually get a nil error because obj.text does not exist

What is the expected output? What do you see instead?

What version of the product are you using?

Do you have an error log of what happened?

Please provide any additional information below.
The only way I have found around this is to create 2 different LDB feeds one for addonloader and one for the mod proper but this obviously isn't great.  Is there some way to make this work that I am missing?

User When Change
Ammo Mar 22, 2011 at 09:43 UTC Changed status from New to Declined
Morgalm Nov 20, 2010 at 06:01 UTC Create

You must login to post a comment. Don't have an account? Register to get one!

  • 4 comments
  • Avatar of mikk mikk Aug 10, 2011 at 22:30 UTC - 0 likes

    @Morgalm: Go "LDB launcher and text field" <- title of ticket. I think he read your ticket ^^

  • Avatar of Morgalm Morgalm Jun 29, 2011 at 20:41 UTC - 0 likes

    It is not incorrect as I am not using a launcher type. Don't think you even read my ticket as I was discussing a "data source" type feed not a launcher.

  • Avatar of Ammo Ammo Mar 22, 2011 at 09:43 UTC - 0 likes

    Your LDB viewer is incorrect. the .text property is not required for "launcher" types.

    https://github.com/tekkub/libdatabroker-1-1/wiki/data-specifications

  • Avatar of Morgalm Morgalm Nov 21, 2010 at 00:38 UTC - 0 likes

    The fix is as simple as this:

    dataobj = LibStub:GetLibrary("LibDataBroker-1.1"):NewDataObject(brokername, {type = "data source", tocname = name, icon = texture, text = "", OnClick = OnClick, OnTooltipShow = OnTooltipShow})
    

    Or you could of course allow authors to choose what type of feed they want "launcher" or "data source". Currently this totally breaks any addon that uses the text field if you use addonloader and LDB loading.

  • 4 comments

Facts

Last updated
Mar 22, 2011
Reported
Nov 20, 2010
Status
Declined - We decided not to take action on this ticket.
Type
Defect - A shortcoming, fault, or imperfection
Priority
Medium - Normal priority.
Votes
0

Reported by

Possible assignees