Missing Image Trouble

RoiEX

While I can't provide a solution, I might be able to present an idea that might help.

I dug through the code and found that there's only a single error message for all the image variations of a unit. Therefore if mech_infantry_hit doesn't exist, you get the same error message as if mech_infantry doesn't exist.

I really have no idea on how to make maps, but I hope this helps

redrum

@beelee Can you upload the whole map zip? Or do you have everything uploaded to the github repo? Just easier than trying to piece together different parts.

beelee

@redrum yea it's at git under "oztea_1939_global"

Thanks guys

Also, although they're not in there for the Americans now, I have tried it with adding the units the new ones replaced into the folder as well.

wc_sumpton

@beelee
Not an expert. But the error shown points to 'mech_infantry' and not 'American_mech_infantry'.

What I tried was to place a copy of the regular units in the Americans folder and a copy of the 'Americans_' units into the other countries folders and the error went away.

Hope this helps...

Cheers...

Cernel

Ok I've put this:
https://github.com/beelee1/oztea_1939_global/archive/master.zip
into my downloadedMaps folder.

Just starting the game, the only missing image I get is:
MISSING IMAGE (this unit or image will be invisible): UnitType{name=American_infantry}

Taking a look at the xml, you have a series of entries, in "unitInitialize", where "American_infantry" is owned by "Russians", "Dutch" and "Mongolians", none of these players having the required image in their units subfolders. This is why you are getting those errors and not seeing those units.

At the developers, I suggest reworking the "Missing Image" error as also to tell the owner of the missing unit. I recall this warning was introduced by @LaFayette.

beelee

@Cernel Right on ! Thanks man ! Yea Idk what happened when I was adding the separate production Frontiers and the new units but the unitInialize was all whacked out and like doubled itself. I got all that sorted but never thought to look close enough at the owners.

Thanks again everyone

redrum

@Cernel Like this?

Cernel

@redrum Yeah, that seems ok. I would write it this way, though, also adding the zone, if this thing was meant to make the game somewhat playable anyways (before, the game just crashed, with any missing images):

MISSING UNIT IMAGE (this unit is not displayed): American_infantry (owner: Russians; zone: Rostov)

Mostly the question here is how it works. If it still takes a placement spot, but you see nothing, like if you would have a blank image, I would say "not visible". If it is not drawn at all on the board, I would say "not displayed".

Looking at the current warning, I guess it works also for missing images that are not units? I mean also something else than "UnitType"? If so, I guess those will need to be reworded, if we want to be specific here.

This would be the current format; not sure why it is worded this way:

MISSING IMAGE (this unit or image will be invisible): UnitType{name=American_infantry}

It makes me think there is a series of other instances of this warning that are not "UnitType". Is this so?

redrum

@Cernel Ah, good idea on location:

Cernel

@redrum If I recall correctly (not sure), correct English doesn't officially allow missing the subject; so it should be "it won't be displayed". If the rest is in that format, I would put the specific elements within quotation marks.

Meaning:

MISSING UNIT IMAGE (it won't be displayed): "American_infantry" owned by "Russians" in "Timguska".

Not sure if the whole thing should have a full stop at the end or just no terminal punctuation at all.

redrum

Good enough. Here is the PR: https://github.com/triplea-game/triplea/pull/4922

Cernel

Should this topic be moved to feature requests, at this point?

RoiEX

@Cernel nah, I think we should be fine

beelee

Well I can take consolation in the fact that my stupidity actually had a beneficial side effect this time :face_with_tears_of_joy: :face_with_tears_of_joy: :face_with_tears_of_joy: