Changeset 13317 in josm


Ignore:
Timestamp:
2018-01-14T01:04:51+01:00 (5 years ago)
Author:
Don-vip
Message:

fix #15755 - Unprocessed HTML in search error box

Location:
trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r13251 r13317  
    420420                        JOptionPane.showMessageDialog(
    421421                                Main.parent,
    422                                 tr("Search expression is not valid: \n\n {0}", e.getMessage()),
     422                                "<html>" + tr("Search expression is not valid: \n\n {0}",
     423                                        e.getMessage().replace("<html>", "").replace("</html>", "")).replace("\n", "<br>") +
     424                                "</html>",
    423425                                tr("Invalid search expression"),
    424426                                JOptionPane.ERROR_MESSAGE);

  • trunk/src/org/openstreetmap/josm/data/osm/search/SearchCompiler.java

    r12666 r13317  
    201201                            return new TimestampRange(minDate, maxDate);
    202202                        } else {
    203                             throw new SearchParseError("<html>" + tr("Expecting {0} after {1}", "<i>min</i>/<i>max</i>", "<i>timestamp</i>"));
     203                            throw new SearchParseError("<html>" + tr("Expecting {0} after {1}", "<i>min</i>/<i>max</i>", "<i>timestamp</i>")
     204                                + "</html>");
    204205                        }
    205206                    }
    206207                } else {
    207                     throw new SearchParseError("<html>" + tr("Expecting {0} after {1}", "<code>:</code>", "<i>" + keyword + "</i>"));
     208                    throw new SearchParseError("<html>" + tr("Expecting {0} after {1}", "<code>:</code>", "<i>" + keyword + "</i>") + "</html>");
    208209                }
    209210            }

  • trunk/test/unit/org/openstreetmap/josm/data/osm/search/SearchCompilerTest.java

    r12659 r13317  
    1818import org.junit.Rule;
    1919import org.junit.Test;
     20import org.junit.rules.ExpectedException;
    2021import org.openstreetmap.josm.TestUtils;
    2122import org.openstreetmap.josm.data.coor.LatLon;
    … …  
    5556    public JOSMTestRules test = new JOSMTestRules().preferences();
    5657
     58    /**
     59     * Rule to assert exception message.
     60     */
     61    @Rule
     62    public ExpectedException expectedEx = ExpectedException.none();
     63
    5764    private static final class SearchContext {
    5865        final DataSet ds = new DataSet();
    … …  
    686693        }
    687694    }
     695
     696    /**
     697     * Non-regression test for <a href="https://josm.openstreetmap.de/ticket/15755">Bug #15755</a>.
     698     * @throws SearchParseError always
     699     */
     700    @Test
     701    public void testTicket15755() throws SearchParseError {
     702        expectedEx.expect(SearchParseError.class);
     703        expectedEx.expectMessage("<html>Expecting <code>:</code> after <i>type</i></html>");
     704        SearchCompiler.compile("public_transport=stop_area -type");
     705    }
    688706}
Note: See TracChangeset for help on using the changeset viewer.