kevinlekiller/eso-linux-launcher

Launcher for Elder Scrolls Online on Linux with a addon updater and a TTC price table updater.

eso-linux-launcher's Issues

Unnecessary "ADIRS" in the addons.sh

I'm not sure what is the purpose of this ADIRS:

ADIRS=""
for dir in $(ls -d "$TMPDIR/"*); do
    dir=$(basename "$dir")
    rm -rf "$1/$dir"
    mv -f "$TMPDIR/$dir" "$1/"
    if [[ $ADIRS == "" ]]; then
        ADIRS="$dir"
    else
       ADIRS="$ADIRS|$dir"
    fi
done
# del url name ver dirs
sed -i "s#$line#$AURI $ANAME $RVERS $ADIRS#" "$CWD/addons.txt"

It seems like all it does is fills the addons.txt with tons of unnecessary clutter text. Every line turns into URL, name, version + very very long directories string that seems to be not needed actually.
I removed it, so it looks like this:

for dir in $(ls -d "$TMPDIR/"*); do
    dir=$(basename "$dir")
    rm -rf "$1/$dir"
    mv -f "$TMPDIR/$dir" "$1/"
done
# del url name ver dirs
sed -i "s#$line#$AURI $ANAME $RVERS#" "$CWD/addons.txt"

and my addont.txt now has just what is actually needed: URL, name, version.

Posting trade information back to TTC

https://us.tamrieltradecentre.com/pc/Trade/WebClient

As seen there, there's an API which receives the information:

static PostTrade(tradeModels) {
	return __awaiter(this, void 0, void 0, function* () {
		if (tradeModels.length == 0)
			return null;
		const resultString = yield HttpService.PostJsonAsync("/api/PC/Trade/PostTrade", tradeModels);
		const bulkPostTradeResult = JSON.parse(resultString);
		return bulkPostTradeResult;
	});
}

[REQUEST] Import HarvestMap files

I'm not very familiar with shell scripting, but it looks like it should be feasible to add in scripting to (optionally) merge HarvestMap data - their powershell script is runnable if you install powershell for linux, but I suspect everything it does is reproducible in a bash script, negating the need to rely on a powershell script.

EU users need to Rename the extracted PriceTable.lua

Maybe i go wrong but i think in the zipped Price Table of the EU users, the file is named PriceTableEU.lua and need to be renamed after extract in PriceTable.lua.

(23) Failed Writing Body

For some addons, I'll get this message "(23) Failed writing body" when updating.

It doesn't seem to necessarily have a negative impact, but I thought it better to report.