Skip to content

Conversation

@tederis
Copy link
Member

@tederis tederis commented Jan 10, 2025

Fixes the return type of server side getColPolygonHeight. Currently it returns a table, whereas two floats are expected.
Additional info: #3537 (comment)

@tederis tederis added the bugfix Solution to a bug of any kind label Jan 10, 2025
@TheNormalnij TheNormalnij added the backwards-incompatible Should be merged after the release of 1.7.1 label Jan 10, 2025
@TheNormalnij TheNormalnij added this to the 1.6.1 milestone Jan 10, 2025
@TheNormalnij
Copy link
Member

Nice catch!
Btw. You need to sync your master with mta-blue master and force-push it to your repo

@tederis tederis force-pushed the GetColPolygonHeight_fix branch from 3f3520c to 7a29cac Compare January 10, 2025 12:52
@botder botder merged commit 0a6b23b into multitheftauto:master Jan 10, 2025
6 checks passed
MTABot pushed a commit that referenced this pull request Jan 10, 2025
0a6b23b Fix return values of getColPolygonHeight on server-side (PR #3946)
018a706 Update client en_US pot
@botder
Copy link
Member

botder commented Jan 10, 2025

I made a mistake here, should've merged this after 1.6.1 release.

@botder botder modified the milestones: 1.6.1, Backlog Jan 10, 2025
botder added a commit that referenced this pull request Jan 10, 2025
botder added a commit that referenced this pull request Jan 10, 2025
@botder botder modified the milestones: Backlog, Next Release (1.6.2) Jan 10, 2025
MTABot pushed a commit that referenced this pull request Jan 10, 2025
f3b3013 Revert "Fix return values of getColPolygonHeight on server-side (PR #3946)"
@Dutchman101
Copy link
Member

As this was reverted, it can now be re-introduced for 1.7 release, so anyone feel free to make a new PR.
Note that the patched function has changed since the revert, so it has to be redone/re-tested by the contributor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backwards-incompatible Should be merged after the release of 1.7.1 bugfix Solution to a bug of any kind

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants