Changeset 4182 in josm


Ignore:
Timestamp:
2011-06-29T09:45:16+02:00 (13 years ago)
Author:
stoecker
Message:

fix #6520 - NPE

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/tagging/TaggingPreset.java

    r4172 r4182  
    279279                if (text != null) {
    280280                    if(text_context != null) {
    281                         locale_text = trc(text_context, text.replaceAll("'","''"));
     281                        locale_text = trc(text_context, fixPresetString(text));
    282282                    } else {
    283                         locale_text = tr(text.replaceAll("'","''"));
     283                        locale_text = tr(fixPresetString(text));
    284284                    }
    285285                }
    … …  
    334334            if(locale_text == null) {
    335335                if(text_context != null) {
    336                     locale_text = trc(text_context, text.replaceAll("'","''"));
     336                    locale_text = trc(text_context, fixPresetString(text));
    337337                } else {
    338                     locale_text = tr(text.replaceAll("'","''"));
     338                    locale_text = tr(fixPresetString(text));
    339339                }
    340340            }
    … …  
    467467                PresetListEntry e = new PresetListEntry(value_array[i]);
    468468                e.display_value = (locale_display_values == null)
    469                 ? (values_context == null ? tr(display_array[i].replaceAll("'","''"))
    470                         : trc(values_context, display_array[i].replaceAll("'","''"))) : display_array[i];
     469                ? (values_context == null ? tr(fixPresetString(display_array[i]))
     470                        : trc(values_context, fixPresetString(display_array[i]))) : display_array[i];
    471471                if (short_descriptions_array != null) {
    472                     e.short_description = locale_short_descriptions == null ? tr(short_descriptions_array[i].replaceAll("'","''"))
    473                             : short_descriptions_array[i].replaceAll("'","''");
     472                    e.short_description = locale_short_descriptions == null ? tr(fixPresetString(short_descriptions_array[i]))
     473                            : fixPresetString(short_descriptions_array[i]);
    474474                }
    475475                lhm.put(value_array[i], e);
    … …  
    524524            if(locale_text == null) {
    525525                if(text_context != null) {
    526                     locale_text = trc(text_context, text.replaceAll("'","''"));
     526                    locale_text = trc(text_context, fixPresetString(text));
    527527                } else {
    528                     locale_text = tr(text.replaceAll("'","''"));
     528                    locale_text = tr(fixPresetString(text));
    529529                }
    530530            }
    … …  
    743743                PresetListEntry e = new PresetListEntry(value_array[i]);
    744744                e.display_value = (locale_display_values == null)
    745                 ? (values_context == null ? tr(display_array[i].replaceAll("'","''"))
    746                         : trc(values_context, display_array[i].replaceAll("'","''"))) : display_array[i];
     745                ? (values_context == null ? tr(fixPresetString(display_array[i]))
     746                        : trc(values_context, fixPresetString(display_array[i]))) : display_array[i];
    747747                if (short_descriptions_array != null) {
    748                     e.short_description = locale_short_descriptions == null ? tr(short_descriptions_array[i].replaceAll("'","''"))
     748                    e.short_description = locale_short_descriptions == null ? tr(fixPresetString(short_descriptions_array[i]))
    749749                            : short_descriptions_array[i];
    750750                }
    … …  
    772772            if (locale_text == null) {
    773773                if(text_context != null) {
    774                     locale_text = trc(text_context, text.replaceAll("'","''"));
     774                    locale_text = trc(text_context, fixPresetString(text));
    775775                } else {
    776                     locale_text = tr(text.replaceAll("'","''"));
     776                    locale_text = tr(fixPresetString(text));
    777777                }
    778778            }
    … …  
    914914            if(locale_text == null) {
    915915                if(text_context != null) {
    916                     locale_text = trc(text_context, text.replaceAll("'","''"));
     916                    locale_text = trc(text_context, fixPresetString(text));
    917917                } else {
    918                     locale_text = tr(text.replaceAll("'","''"));
     918                    locale_text = tr(fixPresetString(text));
    919919                }
    920920            }
    … …  
    937937                    locale_text = tr("More information about this feature");
    938938                } else if(text_context != null) {
    939                     locale_text = trc(text_context, text.replaceAll("'","''"));
     939                    locale_text = trc(text_context, fixPresetString(text));
    940940                } else {
    941                     locale_text = tr(text.replaceAll("'","''"));
     941                    locale_text = tr(fixPresetString(text));
    942942                }
    943943            }
    … …  
    10021002                if (text != null) {
    10031003                    if(text_context != null) {
    1004                         locale_text = trc(text_context, text.replaceAll("'","''"));
     1004                        locale_text = trc(text_context, fixPresetString(text));
    10051005                    } else {
    1006                         locale_text = tr(text.replaceAll("'","''"));
     1006                        locale_text = tr(fixPresetString(text));
    10071007                    }
    10081008                }
    … …  
    11031103        if(locale_name == null) {
    11041104            if(name_context != null) {
    1105                 locale_name = trc(name_context, name.replaceAll("'","''"));
     1105                locale_name = trc(name_context, fixPresetString(name));
    11061106            } else {
    1107                 locale_name = tr(name.replaceAll("'","''"));
     1107                locale_name = tr(fixPresetString(name));
    11081108            }
    11091109        }
    … …  
    14861486    }
    14871487
     1488    private static String fixPresetString(String s) {
     1489        return s == null ? s : s.replaceAll("'","''");
     1490    }
     1491
    14881492    public static Command createCommand(Collection<OsmPrimitive> sel, List<Tag> changedTags) {
    14891493        List<Command> cmds = new ArrayList<Command>();
Note: See TracChangeset for help on using the changeset viewer.