Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
Shadow and Light
Shadow and Light
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 8
    • Issues 8
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Repository
    • Value Stream
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
  • Shadow & Light
  • Shadow and LightShadow and Light
  • Issues
  • #344

Closed
Open
Opened Dec 21, 2019 by Matias@m4tjz

Armory skin enables even when it shouldn't

Description On line 8 of the file ElvUI_SLE\skins\blizzard\character.lua there is an oversight with the return statement causing the Armory skin to be half-enabled and stripping some textures from the Character Frame even when it should be disabled.

The error is on line 8:

if E.db.sle.Armory and not E.db.sle.Armory.Character.Enable then return end

Because the Armory skin was never enabled the value "E.db.sle.Armory" was never set, so it will always return as false, enabling the code to keep going and change textures.

The line should be something like:

if not E.db.sle.Armory or (E.db.sle.Armory and not E.db.sle.Armory.Character.Enable) then return end

Shadow & Light Version 3.63

ElvUI Version 11.27

Steps to Reproduce

  1. Install ElvUI_SLE on a fresh UI without enabling the Armory module.
  2. Should reproduce steps already for Character Frame to be broken.

Expected Behavior Textures should remain if the Skin is disabled

Actual Behavior Some textures of the Character Frame are removed, leaving a mess behind.

ElvUI + S&L Enabled Only ElvUI and ElvUI_SLE enabled.

Only ElvUI Enabled With only ElvUI enabled the Character Frame doesn't get it's textures stripped.

Edited Dec 21, 2019 by Matias
Assignee
Assign to
None
Milestone
None
Assign milestone
Time tracking
None
Due date
None
Reference: shadow-and-light/shadow-and-light#344