Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix message in ter_furn_data don't be translated #66985

Merged
merged 16 commits into from
Jul 20, 2023
Merged

Fix message in ter_furn_data don't be translated #66985

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
5e4abfc
fix ter_furn_data message translation
lispcoc Jul 18, 2023
97fbbc3
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
d0bf4bf
Update src/magic_ter_furn_transform.h
lispcoc Jul 18, 2023
b0ce9b8
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
b295a42
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
72d626e
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
051bbbe
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
0799a35
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
1faa931
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
d75c506
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
5d75bd8
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
8bdb2af
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
0651ed4
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
97d8cce
Update src/magic_ter_fur_transform.cpp
lispcoc Jul 18, 2023
0f25780
optional free
lispcoc Jul 18, 2023
49222c3
Merge branch 'translate_ter_furn_transform' of https://github.com/lis…
lispcoc Jul 18, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 21 additions & 21 deletions src/magic_ter_fur_transform.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ template<class T>
void ter_furn_data<T>::load( const JsonObject &jo )
{
load_transform_results( jo, "result", list );
message = jo.get_string( "message", "" );
jo.read( "message", message, false );
message_good = jo.get_bool( "message_good", true );
}

Expand Down Expand Up @@ -158,7 +158,7 @@ std::optional<ter_furn_data<T>> ter_furn_transform::find_transform( const
}

template<class T, class K>
std::optional<std::pair<T, std::pair<std::string, bool>>> ter_furn_transform::next( const
std::optional<std::pair<T, std::pair<std::optional<translation>, bool>>> ter_furn_transform::next( const
std::map<K, ter_furn_data<T>> &list,
const K &key ) const
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
{
Expand All @@ -170,44 +170,44 @@ std::optional<std::pair<T, std::pair<std::string, bool>>> ter_furn_transform::ne
return std::nullopt;
}

std::optional<std::pair<ter_str_id, std::pair<std::string, bool>>> ter_furn_transform::next_ter(
std::optional<std::pair<ter_str_id, std::pair<std::optional<translation>, bool>>> ter_furn_transform::next_ter(
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const ter_str_id &ter ) const
{
return next( ter_transform, ter );
}

std::optional<std::pair<ter_str_id, std::pair<std::string, bool>>> ter_furn_transform::next_ter(
std::optional<std::pair<ter_str_id, std::pair<std::optional<translation>, bool>>> ter_furn_transform::next_ter(
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const std::string &flag ) const
{
return next( ter_flag_transform, flag );
}

std::optional<std::pair<furn_str_id, std::pair<std::string, bool>>> ter_furn_transform::next_furn(
std::optional<std::pair<furn_str_id, std::pair<std::optional<translation>, bool>>> ter_furn_transform::next_furn(
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const furn_str_id &furn ) const
{
return next( furn_transform, furn );
}

std::optional<std::pair<furn_str_id, std::pair<std::string, bool>>> ter_furn_transform::next_furn(
std::optional<std::pair<furn_str_id, std::pair<std::optional<translation>, bool>>> ter_furn_transform::next_furn(
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const std::string &flag ) const
{
return next( furn_flag_transform, flag );
}

std::optional<std::pair<field_type_id, std::pair<std::string, bool>>>
std::optional<std::pair<field_type_id, std::pair<std::optional<translation>, bool>>>
ter_furn_transform::next_field(
const field_type_id &field ) const
{
return next( field_transform, field );
}

std::optional<std::pair<trap_str_id, std::pair<std::string, bool>>> ter_furn_transform::next_trap(
std::optional<std::pair<trap_str_id, std::pair<std::optional<translation>, bool>>> ter_furn_transform::next_trap(
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const trap_str_id &trap ) const
{
return next( trap_transform, trap );
}

std::optional<std::pair<trap_str_id, std::pair<std::string, bool>>> ter_furn_transform::next_trap(
std::optional<std::pair<trap_str_id, std::pair<std::optional<translation>, bool>>> ter_furn_transform::next_trap(
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const std::string &flag ) const
{
return next( trap_flag_transform, flag );
Expand All @@ -217,25 +217,25 @@ void ter_furn_transform::transform( map &m, const tripoint_bub_ms &location ) co
{
avatar &you = get_avatar();
const ter_id ter_at_loc = m.ter( location );
std::optional<std::pair<ter_str_id, std::pair<std::string, bool>>> ter_potential = next_ter(
std::optional<std::pair<ter_str_id, std::pair<std::optional<translation>, bool>>> ter_potential = next_ter(
ter_at_loc->id );
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const furn_id furn_at_loc = m.furn( location );
std::optional<std::pair<furn_str_id, std::pair<std::string, bool>>> furn_potential = next_furn(
std::optional<std::pair<furn_str_id, std::pair<std::optional<translation>, bool>>> furn_potential = next_furn(
furn_at_loc->id );
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
const trap_str_id trap_at_loc = m.maptile_at( location ).get_trap().id();
std::optional<std::pair<trap_str_id, std::pair<std::string, bool>>> trap_potential = next_trap(
std::optional<std::pair<trap_str_id, std::pair<std::optional<translation>, bool>>> trap_potential = next_trap(
trap_at_loc );
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved

const field &field_at_loc = m.field_at( location );
for( const auto &fld : field_at_loc ) {
std::optional<std::pair<field_type_id, std::pair<std::string, bool>>> field_potential = next_field(
std::optional<std::pair<field_type_id, std::pair<std::optional<translation>, bool>>> field_potential = next_field(
fld.first );
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
if( field_potential ) {
m.add_field( location, field_potential->first, fld.second.get_field_intensity(),
fld.second.get_field_age(), true );
m.remove_field( location, fld.first );
if( you.sees( location ) ) {
you.add_msg_if_player( field_potential->second.first,
if( you.sees( location ) && field_potential->second.first.has_value() ) {
you.add_msg_if_player( field_potential->second.first.value().translated(),
field_potential->second.second ? m_good : m_bad );
}
}
Expand Down Expand Up @@ -279,21 +279,21 @@ void ter_furn_transform::transform( map &m, const tripoint_bub_ms &location ) co

if( ter_potential ) {
m.ter_set( location, ter_potential->first );
if( you.sees( location ) ) {
you.add_msg_if_player( ter_potential->second.first, ter_potential->second.second ? m_good : m_bad );
if( you.sees( location ) && ter_potential->second.first.has_value() ) {
you.add_msg_if_player( ter_potential->second.first.value().translated(), ter_potential->second.second ? m_good : m_bad );
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
}
}
if( furn_potential ) {
m.furn_set( location, furn_potential->first );
if( you.sees( location ) ) {
you.add_msg_if_player( furn_potential->second.first,
if( you.sees( location ) && furn_potential->second.first.has_value() ) {
you.add_msg_if_player( furn_potential->second.first.value().translated(),
furn_potential->second.second ? m_good : m_bad );
}
}
if( trap_potential ) {
m.trap_set( location, trap_potential->first );
if( you.sees( location ) ) {
you.add_msg_if_player( trap_potential->second.first,
if( you.sees( location ) && trap_potential->second.first.has_value() ) {
you.add_msg_if_player( trap_potential->second.first.value().translated(),
trap_potential->second.second ? m_good : m_bad );
}
}
Expand Down
20 changes: 10 additions & 10 deletions src/magic_ter_furn_transform.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ class ter_furn_data
{
public:
weighted_int_list<T> list;
std::string message;
std::optional<translation> message;
lispcoc marked this conversation as resolved.
Show resolved Hide resolved
bool message_good = false;
ter_furn_data() = default;
ter_furn_data( const weighted_int_list<T> &list, const std::string &message,
ter_furn_data( const weighted_int_list<T> &list, const std::optional<translation> &message,
const bool message_good ) :
list( list ), message( message ), message_good( message_good ) {}

Expand All @@ -47,19 +47,19 @@ class ter_furn_transform

std::map<field_type_id, ter_furn_data<field_type_id>> field_transform;

std::optional<std::pair<ter_str_id, std::pair<std::string, bool>>> next_ter(
std::optional<std::pair<ter_str_id, std::pair<std::optional<translation>, bool>>> next_ter(
const ter_str_id &ter ) const;
std::optional<std::pair<ter_str_id, std::pair<std::string, bool>>> next_ter(
std::optional<std::pair<ter_str_id, std::pair<std::optional<translation>, bool>>> next_ter(
const std::string &flag ) const;
std::optional<std::pair<furn_str_id, std::pair<std::string, bool>>> next_furn(
std::optional<std::pair<furn_str_id, std::pair<std::optional<translation>, bool>>> next_furn(
const furn_str_id &furn ) const;
std::optional<std::pair<furn_str_id, std::pair<std::string, bool>>> next_furn(
std::optional<std::pair<furn_str_id, std::pair<std::optional<translation>, bool>>> next_furn(
const std::string &flag ) const;
std::optional<std::pair<trap_str_id, std::pair<std::string, bool>>> next_trap(
std::optional<std::pair<trap_str_id, std::pair<std::optional<translation>, bool>>> next_trap(
const trap_str_id &trap ) const;
std::optional<std::pair<trap_str_id, std::pair<std::string, bool>>> next_trap(
std::optional<std::pair<trap_str_id, std::pair<std::optional<translation>, bool>>> next_trap(
const std::string &flag ) const;
std::optional<std::pair<field_type_id, std::pair<std::string, bool>>> next_field(
std::optional<std::pair<field_type_id, std::pair<std::optional<translation>, bool>>> next_field(
const field_type_id &field )
const;

Expand All @@ -68,7 +68,7 @@ class ter_furn_transform
const K &key ) const;

template <class T, class K>
std::optional<std::pair<T, std::pair<std::string, bool>>> next( const std::map<K, ter_furn_data<T>>
std::optional<std::pair<T, std::pair<std::optional<translation>, bool>>> next( const std::map<K, ter_furn_data<T>>
&list,
const K &key ) const;
lispcoc marked this conversation as resolved.
Show resolved Hide resolved

Expand Down