Proposal: "scenes" key in modinfo.json

Discussion in 'Mod Discussions' started by wondible, March 1, 2014.

  1. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    So I recently saw that the mod ListGenerator had a list of scenes, which makes at least three programs (two versions of PAMM) which have this built in. Would it be better to have a "scenes" object to clearly delineate this information? We'd still need a list for backwards compatibility, but at least that list could be frozen at the current set of scenes.

    EDIT: Example for clarity

    Code:
    {
      "author": "wondible",
      ...
      "scenes": {
        "live_game": [
          "coui://ui/mods/....css",
          "coui://ui/mods/....js"
        ]
      },
      ...
    }
    Raevn, ORFJackal and LavaSnake like this.
  2. cptconundrum

    cptconundrum Post Master General

    Messages:
    4,186
    Likes Received:
    4,900
    That's what we already have, isn't it?
  3. LavaSnake

    LavaSnake Post Master General

    Messages:
    1,620
    Likes Received:
    691
    That's a good idea. Should make updates like this much easier.
  4. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    What we have:

    Code:
    {
      "author": "wondible",
      ...
      "live_game": [
        "coui://ui/mods/....css",
        "coui://ui/mods/....js"
      ],
      ...
    }
  5. proeleert

    proeleert Post Master General

    Messages:
    1,681
    Likes Received:
    1,656
    Ah yes good idea.
  6. DeathByDenim

    DeathByDenim Post Master General

    Messages:
    4,328
    Likes Received:
    2,125
    Yeah, that would help too against having to update PAMM every time PA releases new scenes.
    Raevn likes this.
  7. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    Great idea - added :)
  8. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    As in PAMM-Windows will use the key if it's there?
  9. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    In the next version, yes (It can use either system).
  10. DeathByDenim

    DeathByDenim Post Master General

    Messages:
    4,328
    Likes Received:
    2,125
    Ooh, I feel pressured now. ;)
    LavaSnake and Raevn like this.
  11. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    Does the Windows version at least prompt for program updates, so we could say there is a point at which it is safe to remove the top-level scene names?
  12. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    So "in the next version" was code for "in a few minutes" ? ;^)
    cptconundrum, LavaSnake and Raevn like this.
  13. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    If you mean prompt for new versions of PAMM, then yes (as you probably just found out ;) ). As for when to stop supporting the top-level scene names, I plan to keep the current ones in, but not add any more. If we get to a point where none of the older mods use the top-level names anymore, then I'll remove them, but it's not a big deal to keep them there.
  14. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    I've updated the amdclean template to use scenes as primary and copy them down to the main object during build.
  15. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    I'm on OS X, which is why I had to ask.

    I was referring to mod authors removing the keys, since they will only want to maintain one set.
  16. DeathByDenim

    DeathByDenim Post Master General

    Messages:
    4,328
    Likes Received:
    2,125
    Ok, pamm for Linux/Mac is updated too. Got nuts with the new "scenes" key.
    Raevn likes this.
  17. wondible

    wondible Post Master General

    Messages:
    3,315
    Likes Received:
    2,089
    Does ListGenerator need to be updated?
  18. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    No; the online mod list doesn't have scene information as it isn't needed.
  19. proeleert

    proeleert Post Master General

    Messages:
    1,681
    Likes Received:
    1,656
    Updated ListGen to not copy scenes to modlist.json
  20. Raevn

    Raevn Moderator Alumni

    Messages:
    4,226
    Likes Received:
    4,324
    I stand corrected ;)

Share This Page