Allow allocating new object models using engineRequestModel

[TheNormalnij]

Use first argument object in engineRequestModel to allocate new model. New model will based on trashcan (1337).
Changed valid objects list and this can cause bugs for old scripts. For example, you can't create object with ID 1. New method think that it's a ped.
Valid objects are here

API:

engineRequestModel( "object" )

TEST:

1. Start test resource
2. Press "home" button.
3. This part of map use new allocated ID's

[qaisjp]

Changed valid objects list and this can cause bugs for old scripts.

If this is a separate bugfix, please can you submit it in a separate pull request?

[qaisjp]

Excellent work, by the way :)

[TheNormalnij]

I can't. Without this fix you can't create new objects and change models and textures.

[ghost]

Thanks for the PR. I skimmed through the code, and it looks good. Testing the PR is going to take some time. I suggest we merge this after 1.5.8.

[TheNormalnij]

Thanks you for help, @PlatinMTA !

1. Crash is fixed.
2. I can't reproduce.
3. It's a list with all default object ID's, engineRequestModel will select another ID's.

[TheNormalnij]

Added test resource

[Inder00]

It looks like good

[qaisjp]

Huh interesting, does gta skip over = 2?

[TheNormalnij]

GTA SA don't use model info with enum 2

[TheNormalnij]

Or this is special type for parts of destroyed car

[TheNormalnij]

Fixed formatting

[Pirulax]

@saml1er #1684 (comment)
It is needed, otherwise his range based for loop woudn't work (with CMappedArray)

[ghost]

@saml1er #1684 (comment)
It is needed, otherwise his range based for loop woudn't work (with CMappedArray)

In my other review comment, I suggested him to move the code in CClientPedManager. In this way, the method will become shorter, readable, and this won't be needed.
Hmm, I think I get it now why he added it there. Okay.