Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9.1
    • Component/s: Core & storage
    • Labels:
      None

      Description

      As talked in http://s.apache.org/IP1, TranslatorReader was deprecated in 2.3, and it isn't used in src, so it should be safe to drop it for 2.9.1

        Activity

        Juan Pablo Santos Rodríguez created issue -
        Hide
        Harry Metske added a comment -

        +1

        Show
        Harry Metske added a comment - +1
        Hide
        Juan Pablo Santos Rodríguez added a comment -

        dropped in 2.9.1-svn-28

        Show
        Juan Pablo Santos Rodríguez added a comment - dropped in 2.9.1-svn-28
        Juan Pablo Santos Rodríguez made changes -
        Field Original Value New Value
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Jürgen Weber added a comment -

        There are still several refences in comments:

        ./org/apache/wiki/xmlrpc/RPCHandler.java: // from the TranslatorReader itself. Also, the link format should probably
        ./org/apache/wiki/search/SearchMatcher.java: * TranslatorReader rewrite will presumably invalidate this, among other things.
        ./org/apache/wiki/plugin/Image.java:// do this cleanly, a TranslatorReader revamp is needed.
        ./org/apache/wiki/plugin/AbstractReferralPlugin.java: // Make a Wiki markup link. See TranslatorReader.
        ./org/apache/wiki/plugin/Denounce.java: // FIXME: Should really call TranslatorReader
        ./org/apache/wiki/parser/JSPWikiMarkupParser.java: throw new InternalWikiException("Faulty camelcasepattern in TranslatorReader");
        ./org/apache/wiki/parser/JSPWikiMarkupParser.java: log.warn("TranslatorReader got a faulty page name!",e);
        ./org/apache/wiki/parser/JSPWikiMarkupParser.java: * where HTMLRenderer would try and boot a TranslatorReader before
        ./org/apache/wiki/parser/JSPWikiMarkupParser.java: * the TranslatorReader it is contained by is up.
        ./org/apache/wiki/event/PageEventFilter.java: * but not yet been sent through the TranslatorReader. Note that you cannot
        ./org/apache/wiki/event/PageEventFilter.java: * do HTML translation here, because TranslatorReader is likely to escape it.
        ./org/apache/wiki/event/PageEventFilter.java: * This method is called after a page has been fed through the TranslatorReader,

        Show
        Jürgen Weber added a comment - There are still several refences in comments: ./org/apache/wiki/xmlrpc/RPCHandler.java: // from the TranslatorReader itself. Also, the link format should probably ./org/apache/wiki/search/SearchMatcher.java: * TranslatorReader rewrite will presumably invalidate this, among other things. ./org/apache/wiki/plugin/Image.java:// do this cleanly, a TranslatorReader revamp is needed. ./org/apache/wiki/plugin/AbstractReferralPlugin.java: // Make a Wiki markup link. See TranslatorReader. ./org/apache/wiki/plugin/Denounce.java: // FIXME: Should really call TranslatorReader ./org/apache/wiki/parser/JSPWikiMarkupParser.java: throw new InternalWikiException("Faulty camelcasepattern in TranslatorReader"); ./org/apache/wiki/parser/JSPWikiMarkupParser.java: log.warn("TranslatorReader got a faulty page name!",e); ./org/apache/wiki/parser/JSPWikiMarkupParser.java: * where HTMLRenderer would try and boot a TranslatorReader before ./org/apache/wiki/parser/JSPWikiMarkupParser.java: * the TranslatorReader it is contained by is up. ./org/apache/wiki/event/PageEventFilter.java: * but not yet been sent through the TranslatorReader. Note that you cannot ./org/apache/wiki/event/PageEventFilter.java: * do HTML translation here, because TranslatorReader is likely to escape it. ./org/apache/wiki/event/PageEventFilter.java: * This method is called after a page has been fed through the TranslatorReader,
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        8d 3h 9m 1 Juan Pablo Santos Rodríguez 18/Feb/13 23:05

          People

          • Assignee:
            Unassigned
            Reporter:
            Juan Pablo Santos Rodríguez
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development