DinoDevs / GladiatusCrazyAddon

This was "A browser addon for the Gladiatus browser game."
https://gladiatus.dinodevs.com
GNU General Public License v3.0
41 stars 32 forks source link

Gladiatus Update 4.3.1 Bugs #382

Closed FrutyX closed 2 years ago

FrutyX commented 2 years ago

Gameforge moved their images to a new URL location, so images GCA is using are not being loaded and that is causing a mess.

Reports are failing to detect enemies URL:

🟢 Status: FIXED

`Failed to detect enemy url("//gf3.geo.gfsrv.net/cdn29/bac7e539d61a9de6caec21363c2694.jpg")`

Mystery box rubies prices unknown:

🟢 Status: FIXED

![rubiesprices](https://user-images.githubusercontent.com/23278552/180775996-c353061a-8192-4580-a43e-dac693b2164b.PNG)

Pantheon quests bugged:

🟢 Status: FIXED

``` pantheon.quests.js?4.3.6&built=1658150383374:82 Uncaught TypeError: Cannot read properties of null (reading '1') at Object.quests_reorder (pantheon.quests.js?4.3.6&built=1658150383374:82:122) at Object.inject (pantheon.quests.js?4.3.6&built=1658150383374:15:9) at fireLoad (pantheon.quests.js?4.3.6&built=1658150383374:180:23) ```

Guild battle report bugged:

🟢 Status: FIXED

![guildmsg](https://user-images.githubusercontent.com/23278552/179835127-1ccbd308-5c80-4614-924e-89493cb45435.PNG)

Add to Target List bugged:

🟢 Status: FIXED

![targetlist](https://user-images.githubusercontent.com/23278552/179520794-526ad0e3-c819-45b5-a17f-7346cc077ca0.PNG)

Gold&XP Statistics Icon bugged:

🟢 Status: FIXED

![statistics](https://user-images.githubusercontent.com/23278552/179521020-cdfd4800-66d5-43dc-b0a7-c5c7300332f4.PNG)

Negotium X enemies bugged:

🟢 Status: FIXED

![nego](https://user-images.githubusercontent.com/23278552/179521198-aa849e16-f5cc-4538-aba7-787869503dd5.PNG)

Buffs bugged when viewing other player profiles:

🟢 Status: FIXED

![buffs](https://user-images.githubusercontent.com/23278552/179521951-a67a8b04-8ed6-4b03-8306-33b724d715c0.PNG)

CT and Dungeons table overview bugged:

🟢 Status: FIXED

![ct](https://user-images.githubusercontent.com/23278552/179524567-f0e90b05-8ecf-4a88-ad6b-fffdbf6350f1.PNG) ``` Failed to load resource: the server responded with a status of 404 (Not Found) char_status_kopf_b.jpg:1 Failed to load resource: the server responded with a status of 404 (Not Found) char_status_abschluss_b.jpg:1 ```

Durability loss numbers are bugged with GCA enabled:

🟢 Status: FIXED

![numbersbug](https://user-images.githubusercontent.com/23278552/179643570-05403b5d-9be8-4c6c-af71-0a3bec7b5e57.png)

Attack guild button missing in War Master Hall:

🟢 Status: FIXED

![attackbutton](https://user-images.githubusercontent.com/23278552/179741576-0d910779-12d8-40e9-895b-53469368ff1c.PNG)

Library level indicator bugged:

🟢 Status: FIXED

![librarylevels](https://user-images.githubusercontent.com/23278552/180639241-d78d56f9-9f79-498a-b2ac-ac1a26070396.PNG)

... more will be added when found.

GramThanos commented 2 years ago

https://gf1.geo.gfsrv.net/cdnf8/78e9dd30a5e0f07d5be66c9bae9dd4.jpg https://gf3.geo.gfsrv.net/cdn29/bac7e539d61a9de6caec21363c2694.jpg

These 2 are already in the data.

FrutyX commented 2 years ago

And should this "failed" error occur, with data added?

GramThanos commented 2 years ago

If you have the last version of the extension I guess no.

FrutyX commented 2 years ago

I thought so, and yet, I am still getting these errors.

GramThanos commented 2 years ago

I thought so, and yet, I am still getting these errors.

Both links return a value, thus you shouldn't get an error.

gca_tools.img.resolve('https://gf1.geo.gfsrv.net/cdnf8/78e9dd30a5e0f07d5be66c9bae9dd4.jpg')
gca_tools.img.resolve('https://gf3.geo.gfsrv.net/cdn29/bac7e539d61a9de6caec21363c2694.jpg')
FrutyX commented 2 years ago

I have my suspicion, I'll be back when I confirm or deny it.

FrutyX commented 2 years ago

Confirmed, a secondary drop is causing this errors (very #368 related):

Failed to detect enemy url("//gf3.geo.gfsrv.net/cdn29/bac7e539d61a9de6caec21363c2694.jpg")

scd

FrutyX commented 2 years ago

Even the symbols came back for this one!

symbols

FrutyX commented 2 years ago

You need this for the list?

https://gf3.geo.gfsrv.net/cdn8b/84fd3f76ff45db7d2cdbf5680aa09b.png

FrutyX commented 2 years ago

Helpful?

https://gf3.geo.gfsrv.net/cdnb8/86e9550484e557ea7b3e0b4a825490.jpg https://gf2.geo.gfsrv.net/cdn4a/19e794899bda1deaf03b3339a2e6f3.jpg

FrutyX commented 2 years ago

Confirmed, a secondary drop is causing this errors :

And it does look like, that the secondary drop has to be a material, for this error to happen.

FrutyX commented 2 years ago

So, a recommended To-Do List before a version launch (only things I am currently aware of):

Time to time I get some random error, harmless ones I would say, unfortunately my browser dumped my error logs, probably because they were overspammed, but like I said, they are random and harmless with no priority, I haven't noticed anything that would break the extension, they can also appear now thanks to the loading times, which is absolutely understandable.

FrutyX commented 2 years ago

Added by Thanos!

Nile event enemies: ``` Image link "//gf1.geo.gfsrv.net/cdn65/9f45b7fd7ac900004dcafc2feec57d.jpg" was not found on CDN lookup list. Image link "//gf2.geo.gfsrv.net/cdn10/f0a5434c803bde1ba0913aecd9751b.jpg" was not found on CDN lookup list. Image link "//gf2.geo.gfsrv.net/cdnae/23d6d58d2cf752711ec4a580848476.jpg" was not found on CDN lookup list. Image link "//gf2.geo.gfsrv.net/cdn74/531075909c1e2cb313eda2bc1e8c8e.jpg" was not found on CDN lookup list ```
FrutyX commented 2 years ago

I think I have done everything I wanted to do, everything is currently in the testing phase, I may do some little enhancements, bug fixing and this kind of stuff and the more time we have for testing, the better, however, the biggest question of all questions, when we are going to release the new version? The wolf is already knocking at the door, it's almost about time.

The current problems are mentioned above, but these are mostly between us, things people probably will not even notice.

FrutyX commented 2 years ago

I am off to sleep, you take the wheel Thanos, improve but just don't break anything, I worked hard on it as an javascript amateur :D.

FrutyX commented 2 years ago
  • Last but not least there is the mystery box showing weird ruby values on some items, if you want to check that too

Yo what, I just had a look, you mind explaining to me why you put some weird values on it, before I'll edit it so it makes sense?

Like why for example "hamper" : {rubies : 1.07}, (cervisia) has such value? It's an item that cannot be bought, so you can't put a price tag on it, same goes for 100% healing potion (but we could use the price of Centurion activation, because you get it with it).

Someone has been lazy! :D

FrutyX commented 2 years ago

Alright, they have to be placeholders, right?

FrutyX commented 2 years ago

Well, I would say that's another thing solved.

GramThanos commented 2 years ago

If something can not by bought I suggest to show nothing. We dont have to add more translations. (In greek the unbuyable is "Δεν μπορεί να αγοραστεί" or something like "μη εξαγοράσιμο" ... I would like to see you fitting that in the column).

Regarding the 1.07: you can buy 14 days centurion for 15 rubies thus 1.0714285714285714 rubies per day. Thus 1 cervicia (24hours) could be 1.07 rubies.

FrutyX commented 2 years ago

Jesus christ, Greek problems... everything so long.

And about these mathematics... I really don't think this is needed, interesting to know but a little useless, you cannot buy 1 day cent anyway. Sometimes it's better to go simple.

GramThanos commented 2 years ago

Ideally the rubies numbers are for people wanting to understand what the gain is from the rewards and help them assess whether it make sense to refresh or not. Further more one could also aggregate the rubies values on the rewards and open only boxes with high gain. But for this to be done, the user's opinion on what exactly they are interesting in is more important.

FrutyX commented 2 years ago

xd

Fine, Fine, if you like it so much, do you have a revert commits button there? Feel free to revert it, if not, I'll change it.

There was also Mobilization and Gate Key for 3 rubies (can't buy), magical bag for -1 (can't buy) and costumes for 50 (can't buy).

FrutyX commented 2 years ago

Should've explained this to me 24 or 17 days ago, when I was mentioning it, and not when I make changes to it...

Thanos The Troll.

GramThanos commented 2 years ago

Look, I just say my opinion, I am not here to revert changes. If you think this should change maybe it should.

I argue so that we can reach a consensus. Thus, if you still think they are not needed, we can remove them.

FrutyX commented 2 years ago

I have made some changes.

FrutyX commented 2 years ago

By the way, this is most likely a bug, and costumes are not recognized even with the value, shown as "?".

FrutyX commented 2 years ago

Image link "https://gf3.geo.gfsrv.net/cdnba/fb5aa808eb79bf95e982817b87ff72.jpg" was not found on CDN lookup list.

But I added it on the list? https://github.com/DinoDevs/GladiatusCrazyAddon/commit/6426fea3e8be3858e4b190afa897ff8c0e44acb1

'img/costumes/sets/male/{pluto-we-dont-know-this}.jpg' : 'fb5aa808eb79bf95e982817b87ff72', // Pluto

I tried refreshing the addon.

GramThanos commented 2 years ago

There is a ba missing from the string. It was supposed to be:

'img/costumes/sets/male/{pluto-we-dont-know-this}.jpg' : 'bafb5aa808eb79bf95e982817b87ff72', // Pluto

FrutyX commented 2 years ago

Ah so, didn't know that.

GramThanos commented 2 years ago

Ah so, didn't know that.

i probably forgot to copy the 2 chars next to the '/cdn'

FrutyX commented 2 years ago

Well, with this fixed, if this is supposed to show the costume prices, it doesn't work, not related to the cdn link.

    // game/assets/img/costumes/sets/male/7_complete_small.jpg
            costumes : {rubies : 7}
        },
FrutyX commented 2 years ago

Thanos, I would just like to remind you, that people that don't use the dev version (so 99.9% GCA users), are having a broken extension, I hate pushing and annoying you, but it's maybe time... I am not seeing anything with high priority that is a problem, the mystery chest does not show costumes ruby values which is a very small bug and the failed to detect enemy error, well, that is happening only on a secondary material drop, and I've checked the reports and the drop is normally saved, so it's basically only a console warning, but it does not really break the extension in any way.

Other than that, I haven't even detected any other errors, of course there is always still room for improvements, bug fixing and new features, but like this, we would never release a new version.

GramThanos commented 2 years ago

Hey. I used the flows I scripted a few months ago to push the versions. I think they run correctly. Lets see.

GramThanos commented 2 years ago

Update, I got an email from Firefox that the version is live. Wrong, it is the offline version.

FrutyX commented 2 years ago

kraken

FrutyX commented 2 years ago

It seems it's live on Firefox store.

FrutyX commented 2 years ago

Okie, I moved everything relevant to it's own issues, and this seems to be completed.

From what I know from Apo (about Thanos), I will keep the finished issues open for one or two weeks after the release (in case there are any problems) and then close them all, for moderation purposes.

GramThanos commented 2 years ago

Purge everything.

FrutyX commented 2 years ago

I love purging!

FrutyX commented 2 years ago

Patched in v4.3.6, closing, re-open if needed.