iopietro / Travianz-Legacy

Join our Discord Server: https://discordapp.com/invite/9fbJKP9 | New repo: https://github.com/iopietro/Travianz
GNU General Public License v3.0
160 stars 94 forks source link

For developers and testers. 8.3.3 version #518

Closed Shadowss closed 6 years ago

Shadowss commented 6 years ago

We must a full evaluation of the entire version and put all bugs in 1 posts. To see what bugs is still remaining and what need testings.

And other bugs I didn't find yet or that I forgot.

iopietro commented 6 years ago

@fritz-net Fixed in bd38ac7fd8861cbcd46348ed386c1ccc8d8420c7.

eliopinho commented 6 years ago

@iopietro can you group reenforcements at rally point ?

if you send multiplous reenforcements from the same village , should group all troops in only one.. (even the reports , when the troops are attacked, you receive multiplous reports..)

should only receive one report per troops and village.

iopietro commented 6 years ago

@eliopinho, Fixed in bf091c2a0ce917cbc1fffcb5eeaca11365fb01ea.

eliopinho commented 6 years ago

thks a lot

Shadowss commented 6 years ago

Great , thanks a lot for all. We are almost there , we are a half road on that section.

Maybe you can try to fix General building bugs and Artefacts next. But you know better what is next. It`s just an ideea

iopietro commented 6 years ago

First I'll rework the report system, there are too many duplicate files in the notice folder, I'll delete about 80% of them. Then I'll move into fixing artefacts and after general building bugs.

Shadowss commented 6 years ago

Huston we have a problem.

I attack @velhbxtyrj World of Wonder , the reports is correct but not counted in TOP 10.

report top10

Shadowss commented 6 years ago

Another problem in PARADISE , when hero died , and enter in Admin panel to revive in here :

hero revive

And press revive , nothing is happened. Hero must to revive in the CAPITAL.

velhbxtyrj commented 6 years ago

Not included are attack points

default

Shadowss commented 6 years ago

New bug added in Artifact Section :

Any player may only capture 1 artefact with an unique or account scope. Any player may only capture up to 3 artefacts "at once". Players may own more than one unique/account artefact resp. more than 3 artefacts in total by conquering. Only 3 artefacts are active out of which only 1 can be unique or have an account wide effect. In case there are 2 account wide effect artefacts amongst your 3 oldest artefacts the fourth oldest will become active. If player has 3 village scope artefacts and conquers a fourth with an account scope effect, the third (most recent) village scope artefact gets deactivated, and instead of it the fourth artefact will get active after the countdown.

treasury

@velhbxtyrj : maybe you update the list with thees bugs please (exclude Artifact Bug , i already put it on first post). Thanks

iopietro commented 6 years ago

I'll look into it, the problem with Automation.php (and in particular, the function "sendunitsComplete") is that it's very huuuuuge. Automation.php has about 7k lines of code and sendunitsComplete 2k, that's very bad because the maintanability is very low, so if I change something significant in the code, I risk to break everything, lol.

velhbxtyrj commented 6 years ago

Need to look in your last change. This is due to reports. I just reloaded all the old files on the hosting and everything works :)

iopietro commented 6 years ago

Yep, I fixed it internally, I'll push the commit later today, with the report refactor part 2.

iopietro commented 6 years ago

Fixed in c2f4d43b7ef435133ef67ae90df09fd988ed7902.

velhbxtyrj commented 6 years ago

During the attack on Natars. This is due to the destruction of targets by catapults.

Warning: Illegal string offset 'userid' in /home/u997437127/public_html/GameEngine/Automation.php on line 5119

Warning: Illegal string offset 'oldrank' in /home/u997437127/public_html/GameEngine/Automation.php on line 5121

Warning: Illegal string offset 'userid' in /home/u997437127/public_html/GameEngine/Automation.php on line 5122

Warning: Illegal string offset 'oldrank' in /home/u997437127/public_html/GameEngine/Automation.php on line 5131

Warning: Illegal string offset 'userid' in /home/u997437127/public_html/GameEngine/Automation.php on line 5132

Warning: Illegal string offset 'oldrank' in /home/u997437127/public_html/GameEngine/Automation.php on line 5121

Warning: Illegal string offset 'userid' in /home/u997437127/public_html/GameEngine/Automation.php on line 5122

Warning: Illegal string offset 'oldrank' in /home/u997437127/public_html/GameEngine/Automation.php on line 5131

Warning: Illegal string offset 'userid' in /home/u997437127/public_html/GameEngine/Automation.php on line 5132

Warning: Illegal string offset 'oldrank' in /home/u997437127/public_html/GameEngine/Automation.php on line 5121

Warning: Illegal string offset 'userid' in /home/u997437127/public_html/GameEngine/Automation.php on line 5122

Warning: Illegal string offset 'oldrank' in /home/u997437127/public_html/GameEngine/Automation.php on line 5131

Warning: Illegal string offset 'userid' in /home/u997437127/public_html/GameEngine/Automation.php on line 5132

default

iopietro commented 6 years ago

Should be fixed in 970387342a7f8603992ed8709dfae3d13fab0e05.

Another thing, I don't think that "duplicated reports in the alliance" is a bug, because if i remember correctly it was present in the original travian.

Shadowss commented 6 years ago

Yes it's in the original version

iopietro commented 6 years ago

Ok, so I'll remove it from the bug list.

velhbxtyrj commented 6 years ago

Please watch this video.

iopietro commented 6 years ago

Ok, I miserunderstood, I thought you meant another thing, I'll fix that.

velhbxtyrj commented 6 years ago

I also thought that you thought about where the sword and shield are displayed ) so I decided to show it visually )

iopietro commented 6 years ago

Fixed in 996f0116665ae71e417be2472ec832d10d21bf37.

velhbxtyrj commented 6 years ago

I robbed an oasis and was able to carry out 8,000 of each resource. Must be 800 in fact?

default

iopietro commented 6 years ago

Resources in oasis scale with the "STORAGE_MULTIPLIER" constant.

velhbxtyrj commented 6 years ago

I suspected this) Just thought that it depends on the speed of the server :)

Shadowss commented 6 years ago

Good morning guys.

@iopietro : what's next ?

iopietro commented 6 years ago

Good morning, artifacts are my next target. I need to create a new table in the database for artifacts chronology.

Shadowss commented 6 years ago

@iopietro : don't forget about this : Any player may only capture 1 artefact with an unique or account scope. Any player may only capture up to 3 artefacts "at once". Players may own more than one unique/account artefact resp. more than 3 artefacts in total by conquering. Only 3 artefacts are active out of which only 1 can be unique or have an account wide effect. In case there are 2 account wide effect artefacts amongst your 3 oldest artefacts the fourth oldest will become active. If player has 3 village scope artefacts and conquers a fourth with an account scope effect, the third (most recent) village scope artefact gets deactivated, and instead of it the fourth artefact will get active after the countdown.

velhbxtyrj commented 6 years ago

I would single out this item only. Because the strikethrough already works.

~Any player may only capture 1 artefact with an unique or account scope. Any player may only capture up to 3 artefacts "at once". Players may own more than one unique/account artefact resp. more than 3 artefacts in total by conquering.~ Only 3 artefacts are active out of which only 1 can be unique or have an account wide effect. In case there are 2 account wide effect artefacts amongst your 3 oldest artefacts the fourth oldest will become active. If player has 3 village scope artefacts and conquers a fourth with an account scope effect, the third (most recent) village scope artefact gets deactivated, and instead of it the fourth artefact will get active after the countdown.

Shadowss commented 6 years ago

Wrong

artefact

Unique diet control (50%) Treasury 20, Effect account

Great storage masterplan (GG&GW) Treasury 20, Effect account

Artefact of the unique fool Treasury 20, Effect account

All are active !!

iopietro commented 6 years ago

You've 4 artifacts because you conquered "Ancient Construction plan" before I fixed the possibility to conquer it even with the maximum number of artifacts (3). The artifact activation part isn't in the "barred" part of velhbxtyrj's message.

Shadowss commented 6 years ago

Ahh ok. Must to reinstall all server and restart. But only after you fix all artifacts bug

velhbxtyrj commented 6 years ago

@Shadowss The server does not need to be reinstalled. Just go to another account and try to capture the artifacts. The hero will not be able to capture more than one artifact (large or unique) and in total more than three (one large or unique and two small). The rest of the artifacts can be captured only by capturing the settlement with the located artifact of the leaders. At this stage, just need to fix the work of the captured artifacts as it is painted in the fact that I left it not crossed out :).

fritz-net commented 6 years ago

When I open http://travian.shadowss.ro/berichte.php?t=2 I get an Error 500. When I open the page without the get attributes all is fine. This only happens when I have a lot of reports

agony09 commented 6 years ago

i have try this link. it works fine. no Error 500. is Reports, Trade

iopietro commented 6 years ago

Dunno, it could be a timing out error. @Shadowss You should check your error logs for more infos about that.

fritz-net commented 6 years ago

I don't think its a timeout since its very fast. The link also works fine if I change account

Shadowss commented 6 years ago

Works for me , sorry

Shadowss commented 6 years ago

[23-Apr-2018 19:45:30 Europe/Bucharest] PHP Warning: unlink(GameEngine/Prevention/market.txt): No such file or directory in /home/outlet/public_html/shadowss.ro/travian/GameEngine/Automation.php on line 1007 [23-Apr-2018 20:28:37 Europe/Bucharest] PHP Warning: unlink(GameEngine/Prevention/demolition.txt): No such file or directory in /home/outlet/public_html/shadowss.ro/travian/GameEngine/Automation.php on line 4483 [23-Apr-2018 20:28:37 Europe/Bucharest] PHP Warning: unlink(GameEngine/Prevention/market.txt): No such file or directory in /home/outlet/public_html/shadowss.ro/travian/GameEngine/Automation.php on line 1007 [23-Apr-2018 20:29:08 Europe/Bucharest] PHP Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 20480 bytes) in /home/outlet/public_html/shadowss.ro/travian/GameEngine/Database.php on line 642 [23-Apr-2018 21:49:40 Europe/Bucharest] PHP Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate 20480 bytes) in /home/outlet/public_html/shadowss.ro/travian/GameEngine/Database.php on line 642

iopietro commented 6 years ago

I don't understand what does he mean with this phrase: "In case there are 2 account wide effect artefacts amongst your 3 oldest artefacts the fourth oldest will become active.", if I have 3 artifacts among which 2 unique, how could the fourth artifact activates, if I've not a fourth artifact?

velhbxtyrj commented 6 years ago

Work should only three artifacts, which were captured first. All other artifacts should not work. Now all the artifacts work. You can refer to this help: http://travian.wikia.com/wiki/Artifacts

velhbxtyrj commented 6 years ago

I will also add that only one unique or large artifact should work. But now if we have two or more they all work.

fritz-net commented 6 years ago

@Shadowss, sure it is working for you since you don't have this much reports. I also works for me if I have less (unread) reports. And also if I switch to another account I don't have the issue. Just check your error.log for the 500Error

image

iopietro commented 6 years ago

@velhbxtyrj, Yes, I know, the only problem was in that phrase: If I've 3 artifacts, 2 unique and 1 partial (village effect) which one of the unique should I activate? The oldest one?

velhbxtyrj commented 6 years ago

@iopietro Yes. The oldest (it's the one that we captured the very first).

eliopinho commented 6 years ago

@iopietro after i train some troops the system stay like this..

barracks

and no change since that (feew days ago) on the shadowss server

page reloads every second, if i train troops it finish instantly

i do not know how to reproduce that, or what happen, if something on db or cache :thinking:

iopietro commented 6 years ago

It's already fixed in a newer version of the game, probably shadowss' server doesn't have it yet.

Shadowss commented 6 years ago

Server has been fully update.

velhbxtyrj commented 6 years ago

@iopietro @fritz-net is right. With a large number of unread reports, Error 500 on all tabs except the tab all.

Trade | Reinforcement | Attacks | Miscellaneous | Archive -> Error 500

This I was looking at the server @Shadowss.

A little later I'll check it on my server.

velhbxtyrj commented 6 years ago

@iopietro On my server this does not happen. And judging from the error message it's not enough memory size.