Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use item and use with for guests in houses #1338

Merged
merged 15 commits into from
Aug 8, 2023

Conversation

dudantas
Copy link
Member

@dudantas dudantas commented Aug 2, 2023

Added "ladder" and "dummy" variable in item type to facilitate and improve maintainability of item types that are ladder and dummy.

In addition, some house-related bugs were fixed, such as not being able to use items in the backpack itself (if you are a guest), not being able to use dummy, stairs, and other small problems that interfere with gameplay.

Created a function that exports ladder/dummy IDs to Lua, also facilitating project maintenance, so we don't need hardcoded IDs directly in lua scripts, just define each item in the XML as "ladder" or "dummy", in addition to server for behaviours like of houses...

@dudantas dudantas force-pushed the fix-use-item-in-house-by-guest branch from a8ac5a7 to 64020c2 Compare August 2, 2023 06:10
@paczielny
Copy link

has a problem, potions cannot be used in protection zone

@dudantas
Copy link
Member Author

dudantas commented Aug 5, 2023

has a problem, potions cannot be used in protection zone

#edit
Fixed here: e2023cc

image
image

The pull request only changes the behavior of the houses.

@dudantas dudantas force-pushed the fix-use-item-in-house-by-guest branch from 62fa13d to 0e1fbaf Compare August 7, 2023 13:55
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 6 Code Smells

0.0% 0.0% Coverage
0.3% 0.3% Duplication

@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 8 Code Smells

0.0% 0.0% Coverage
0.3% 0.3% Duplication

@dudantas dudantas merged commit 45f2c57 into opentibiabr:main Aug 8, 2023
@dudantas dudantas deleted the fix-use-item-in-house-by-guest branch September 1, 2023 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants