Rename "material" Brick to "material" Bricks

[Tim79000]

It doesn't make sense, because it isn't one brick, it's a block made of multiple.

[appgurueu]

First of all: Naming is always debatable. From a technical standpoint however, such a rename is not feasible since it would be breaking compatibility. An alias could be used to work around this to some extent, but still wouldn't preserve compatibility e.g. for raw node.name == "..." checks. It's simply not worth the breakage / effort just for "fixing" a name that "doesn't make sense" (to you).

[Tim79000]

I mean, the name that is shown. In the sense that mese blocks used to be called simply "mese"

[Tim79000]

I don't mean the codename, the codename shouldn't get to be changed. Just the displayed name.

[appgurueu]

Ah okay, that would then be a trivial nonbreaking change (that said, we'd be introducing a display name - technical name inconsistency, which I'm not a fan of).

[Naniono]

Ok 2023-02-27, pr 10:35, Lars Müller ***@***.***> rašė:

…

Reopened #3016 <#3016>. — Reply to this email directly, view it on GitHub <#3016 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A6DOUJP4QU7KDDPDDA3MICTWZRYOPANCNFSM6AAAAAAVIVTM7Q> . You are receiving this because you are subscribed to this thread.Message ID: ***@***.***>

[Tim79000]

But it's more of a logical consistency, due to it being more than one brick.

[Tim79000]

Mese blocks also have this inconsistancy, as previously stated.

[Naniono]

---------- Forwarded message --------- Nuo: Tim79000 ***@***.***> Date: 2023-03-02, kt 20:27 Subject: Re: [minetest/minetest_game] Rename "material" Brick to "material" Bricks (Issue #3016) To: minetest/minetest_game ***@***.***> Cc: Naniono ***@***.***>, Comment ***@***.***> Mese blocks also have this inconsistancy, as previously stated. — Reply to this email directly, view it on GitHub <#3016 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A6DOUJOT2BD7FGEKUACJFDTW2DYBTANCNFSM6AAAAAAVIVTM7Q> . You are receiving this because you commented.Message ID: ***@***.***>

[VBPROGER]

We will also need to update translations.