Move more body_part to bodypart_id in Character part1

[Fris0uman]

Summary

SUMMARY: Infrastructure "Move more body_part to bodypart_id in Character part1"

Purpose of change

Part of #39068

Describe the solution

Describe alternatives you've considered

Testing

Compile without error

Additional context

Needs #40138

[anothersimulacrum]

src/bionics_ui.cpp:275:52: error: loop variable 'elem' has type 'const std::pair<bodypart_str_id, size_t> &' (aka 'const pair<string_id<body_part_type>, unsigned long> &') but is initialized with type 'const std::pair<const string_id<body_part_type>, unsigned long>' resulting in a copy [-Werror,-Wrange-loop-construct]
    for( const std::pair<bodypart_str_id, size_t> &elem : bio_id->occupied_bodyparts ) {
                                                   ^
src/bionics_ui.cpp:275:10: note: use non-reference type 'std::pair<bodypart_str_id, size_t>' (aka 'pair<string_id<body_part_type>, unsigned long>') to keep the copy or type 'const std::pair<const string_id<body_part_type>, unsigned long> &' to prevent copying
    for( const std::pair<bodypart_str_id, size_t> &elem : bio_id->occupied_bodyparts ) {
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/bionics_ui.cpp:564:68: error: loop variable 'elem' has type 'const std::pair<bodypart_str_id, size_t> &' (aka 'const pair<string_id<body_part_type>, unsigned long> &') but is initialized with type 'const std::pair<const string_id<body_part_type>, unsigned long>' resulting in a copy [-Werror,-Wrange-loop-construct]
                    for( const std::pair<bodypart_str_id, size_t> &elem : bio_id->occupied_bodyparts ) {
                                                                   ^
src/bionics_ui.cpp:564:26: note: use non-reference type 'std::pair<bodypart_str_id, size_t>' (aka 'pair<string_id<body_part_type>, unsigned long>') to keep the copy or type 'const std::pair<const string_id<body_part_type>, unsigned long> &' to prevent copying
                    for( const std::pair<bodypart_str_id, size_t> &elem : bio_id->occupied_bodyparts ) {
                         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
2 errors generated.
make: *** [Makefile:862: obj/tiles/bionics_ui.o] Error 1
make: *** Waiting for unfinished jobs....
src/bionics.cpp:228:52: error: loop variable 'element' has type 'const std::pair<bodypart_str_id, size_t> &' (aka 'const pair<string_id<body_part_type>, unsigned long> &') but is initialized with type 'const std::pair<const string_id<body_part_type>, unsigned long>' resulting in a copy [-Werror,-Wrange-loop-construct]
    for( const std::pair<bodypart_str_id, size_t> &element : bid->occupied_bodyparts ) {
                                                   ^
src/bionics.cpp:228:10: note: use non-reference type 'std::pair<bodypart_str_id, size_t>' (aka 'pair<string_id<body_part_type>, unsigned long>') to keep the copy or type 'const std::pair<const string_id<body_part_type>, unsigned long> &' to prevent copying
    for( const std::pair<bodypart_str_id, size_t> &element : bid->occupied_bodyparts ) {
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/bionics.cpp:2360:52: error: loop variable 'elem' has type 'const std::pair<bodypart_str_id, size_t> &' (aka 'const pair<string_id<body_part_type>, unsigned long> &') but is initialized with type 'const std::pair<const string_id<body_part_type>, unsigned long>' resulting in a copy [-Werror,-Wrange-loop-construct]
    for( const std::pair<bodypart_str_id, size_t> &elem : bio_id->occupied_bodyparts ) {
                                                   ^
src/bionics.cpp:2360:10: note: use non-reference type 'std::pair<bodypart_str_id, size_t>' (aka 'pair<string_id<body_part_type>, unsigned long>') to keep the copy or type 'const std::pair<const string_id<body_part_type>, unsigned long> &' to prevent copying
    for( const std::pair<bodypart_str_id, size_t> &elem : bio_id->occupied_bodyparts ) {
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
src/bionics.cpp:2389:52: error: loop variable 'elem' has type 'const std::pair<bodypart_str_id, size_t> &' (aka 'const pair<string_id<body_part_type>, unsigned long> &') but is initialized with type 'const std::pair<const string_id<body_part_type>, unsigned long>' resulting in a copy [-Werror,-Wrange-loop-construct]
    for( const std::pair<bodypart_str_id, size_t> &elem : bioid->occupied_bodyparts ) {
                                                   ^
src/bionics.cpp:2389:10: note: use non-reference type 'std::pair<bodypart_str_id, size_t>' (aka 'pair<string_id<body_part_type>, unsigned long>') to keep the copy or type 'const std::pair<const string_id<body_part_type>, unsigned long> &' to prevent copying
    for( const std::pair<bodypart_str_id, size_t> &elem : bioid->occupied_bodyparts ) {
         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
3 errors generated.