Compare commits

...

2 Commits

Author SHA1 Message Date
Florian Brinker
113bc79d77 Update luacheck rules
Some checks failed
continuous-integration/drone/push Build is failing
2019-10-24 18:06:39 +02:00
Florian Brinker
bbfec84081 Fix variable issues 2019-10-24 18:06:26 +02:00
4 changed files with 18 additions and 9 deletions

View File

@ -19,7 +19,8 @@ steps:
- apk add --no-cache build-base curl unzip
- apk add --no-cache lua5.1 lua5.1-dev luarocks5.1
- luarocks-5.1 install luacheck
- luacheck sources/*.lua --compat --allow-defined --no-unused-args --no-max-line-length
- cd sources
- luacheck ./*.lua
- name: build
image: alpine

12
.luacheckrc Normal file
View File

@ -0,0 +1,12 @@
allow_defined = true
max_line_length = false
std = {
read_globals = {
"LINK_STYLE_BRACKET",
"EVENT_MANAGER",
"CHAT_SYSTEM",
"ZO_SavedVars",
"LibStub"
}
}

View File

@ -50,7 +50,6 @@ function AchievementInfo.onAchievementUpdated(eventCode, achId)
local percentageStepSize = AchievementInfo.settingGet("genShowUpdateSteps")
local link = GetAchievementLink(achId, LINK_STYLE_BRACKET)
local name, description = GetAchievementInfo(achId)
local catName = "/"
if categoryId ~= false then
@ -61,14 +60,14 @@ function AchievementInfo.onAchievementUpdated(eventCode, achId)
local numCriteria = GetAchievementNumCriteria(achId)
for i = 1, numCriteria, 1 do
local name, numCompleted, numRequired = GetAchievementCriterion(achId, i)
local description, numCompleted, numRequired = GetAchievementCriterion(achId, i)
local tmpOutput = ""
if i > 1 and AchievementInfo.settingGet("genOnePerLine") == false then
tmpOutput = tmpOutput .. ", "
end
tmpOutput = tmpOutput .. name .. " "
tmpOutput = tmpOutput .. description .. " "
tmpOutput = tmpOutput .. AchievementInfo.calcCriteriaColor(numCompleted, numRequired) .. numCompleted .. "|r"
tmpOutput = tmpOutput .. AchievementInfo.clrDefault .. "/" .. "|r"
tmpOutput = tmpOutput .. AchievementInfo.clrCriteriaComplete .. numRequired .. "|r"
@ -163,13 +162,11 @@ end
-- Get the correct achievement category
function AchievementInfo.getCorrectAchievementCategoryId(achId)
local previousAchievementId = GetPreviousAchievementInLine(achId)
local categoryId = 0
if AchievementInfo.checkForValidCategory(achId) == false and previousAchievementId ~= 0 then
return AchievementInfo.getCorrectAchievementCategoryId(previousAchievementId)
elseif AchievementInfo.checkForValidCategory(achId) then
categoryId = GetCategoryInfoFromAchievementId(achId)
return categoryId
return GetCategoryInfoFromAchievementId(achId)
else
return false
end

View File

@ -122,10 +122,9 @@ function AchievementInfo.createSettingsPanel()
-- Add categories dynamically
local numCats = GetNumAchievementCategories()
local catCount = 1
for i = 1, numCats, 1 do
catName, numSubCats = GetAchievementCategoryInfo(i)
catName = GetAchievementCategoryInfo(i)
table.insert(optionsTable, {
type = "checkbox",