Ignore:
Timestamp:
2013-10-10T17:21:30+13:00 (11 years ago)
Author:
davidb
Message:

Elimination of the 'this.doc' field from the Action baseclass and the subclasses that rely on it. For Greenstone3 purposes it is unsafe to create this object in the constructor to the action and then store it for other methods to access. This is because the Greenstone 3 (and in particular calls to 'process' operate in a multi-threaded context, that is managed by the Servlet server (e.g. Tomcat by default). Calls to DOM methods are not guaranteed to be thread safe, this became apparent when we started looking in to an exception that was being thrown, and centred around use of the DOM method 'item(i)'. The change this commit makes is to remove 'this.doc' being stored as a field. A document is now created in the top level of a call to 'process()' and when a DOM reference is needed in a subsequent method an Element variable (typically passed in as a parameter to the method) is used (through 'Document doc = element.getOwnerDocument()') to gain access to the DOM

File:
1 edited

Legend:

Unmodified
Added
Removed

  • main/trunk/greenstone3/src/java/org/greenstone/gsdl3/action/DocumentAction.java

    r28258 r28382  
    8282
    8383        Element message = this.converter.nodeToElement(message_node);
    84 
     84        Document doc = message.getOwnerDocument();
     85       
    8586        // the response
    86         Element result = this.doc.createElement(GSXML.MESSAGE_ELEM);
    87         Element page_response = this.doc.createElement(GSXML.RESPONSE_ELEM);
     87        Element result = doc.createElement(GSXML.MESSAGE_ELEM);
     88        Element page_response = doc.createElement(GSXML.RESPONSE_ELEM);
    8889        result.appendChild(page_response);
    8990
    … …  
    174175        // the_document is where all the doc info - structure and metadata etc
    175176        // is added into, to be returned in the page
    176         Element the_document = this.doc.createElement(GSXML.DOCUMENT_ELEM);
     177        Element the_document = doc.createElement(GSXML.DOCUMENT_ELEM);
    177178        page_response.appendChild(the_document);
    178179
    179180        // create a basic doc list containing the current node
    180         Element basic_doc_list = this.doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
    181         Element current_doc = this.doc.createElement(GSXML.DOC_NODE_ELEM);
     181        Element basic_doc_list = doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
     182        Element current_doc = doc.createElement(GSXML.DOC_NODE_ELEM);
    182183        basic_doc_list.appendChild(current_doc);
    183184        if (document_id != null)
    … …  
    207208
    208209        // Create a parameter list to specify the required structure information
    209         Element ds_param_list = this.doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
     210        Element ds_param_list = doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
    210211
    211212        if (service_params != null)
    212213        {
    213             GSXML.addParametersToList(this.doc, ds_param_list, service_params);
     214            GSXML.addParametersToList(doc, ds_param_list, service_params);
    214215        }
    215216
    … …  
    223224            if (expand_contents)
    224225            {
    225                 ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     226                ds_param = doc.createElement(GSXML.PARAM_ELEM);
    226227                ds_param_list.appendChild(ds_param);
    227228                ds_param.setAttribute(GSXML.NAME_ATT, "structure");
    … …  
    230231
    231232            // get the info needed for paged naviagtion
    232             ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     233            ds_param = doc.createElement(GSXML.PARAM_ELEM);
    233234            ds_param_list.appendChild(ds_param);
    234235            ds_param.setAttribute(GSXML.NAME_ATT, "info");
    235236            ds_param.setAttribute(GSXML.VALUE_ATT, "numSiblings");
    236             ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     237            ds_param = doc.createElement(GSXML.PARAM_ELEM);
    237238            ds_param_list.appendChild(ds_param);
    238239            ds_param.setAttribute(GSXML.NAME_ATT, "info");
    239240            ds_param.setAttribute(GSXML.VALUE_ATT, "numChildren");
    240             ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     241            ds_param = doc.createElement(GSXML.PARAM_ELEM);
    241242            ds_param_list.appendChild(ds_param);
    242243            ds_param.setAttribute(GSXML.NAME_ATT, "info");
    … …  
    245246            if (get_siblings)
    246247            {
    247                 ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     248                ds_param = doc.createElement(GSXML.PARAM_ELEM);
    248249                ds_param_list.appendChild(ds_param);
    249250                ds_param.setAttribute(GSXML.NAME_ATT, "structure");
    … …  
    257258            if (expand_contents)
    258259            {
    259                 ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     260                ds_param = doc.createElement(GSXML.PARAM_ELEM);
    260261                ds_param_list.appendChild(ds_param);
    261262                ds_param.setAttribute(GSXML.NAME_ATT, "structure");
    … …  
    265266            {
    266267                // get the info needed for table of contents
    267                 ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     268                ds_param = doc.createElement(GSXML.PARAM_ELEM);
    268269                ds_param_list.appendChild(ds_param);
    269270                ds_param.setAttribute(GSXML.NAME_ATT, "structure");
    270271                ds_param.setAttribute(GSXML.VALUE_ATT, "ancestors");
    271                 ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     272                ds_param = doc.createElement(GSXML.PARAM_ELEM);
    272273                ds_param_list.appendChild(ds_param);
    273274                ds_param.setAttribute(GSXML.NAME_ATT, "structure");
    … …  
    275276                if (get_siblings)
    276277                {
    277                     ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     278                    ds_param = doc.createElement(GSXML.PARAM_ELEM);
    278279                    ds_param_list.appendChild(ds_param);
    279280                    ds_param.setAttribute(GSXML.NAME_ATT, "structure");
    … …  
    292293
    293294            // Build a request to obtain the document structure
    294             Element ds_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
     295            Element ds_message = doc.createElement(GSXML.MESSAGE_ELEM);
    295296            String to = GSPath.appendLink(collection, "DocumentStructureRetrieve");// Hard-wired?
    296             Element ds_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
     297            Element ds_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
    297298            ds_message.appendChild(ds_request);
    298299            ds_request.appendChild(ds_param_list);
    … …  
    316317            if (ds_response_struct_info != null)
    317318            {
    318                 the_document.appendChild(this.doc.importNode(ds_response_struct_info, true));
     319                the_document.appendChild(doc.importNode(ds_response_struct_info, true));
    319320            }
    320321            path = GSPath.appendLink(GSXML.RESPONSE_ELEM, GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
    … …  
    329330                for (int i = 0; i < structs.getLength(); i++)
    330331                {
    331                     the_document.appendChild(this.doc.importNode(structs.item(i), true));
     332                    the_document.appendChild(doc.importNode(structs.item(i), true));
    332333                }
    333334            }
    … …  
    335336            {
    336337                // no structure nodes, so put in a dummy doc node
    337                 Element doc_node = this.doc.createElement(GSXML.DOC_NODE_ELEM);
     338                Element doc_node = doc.createElement(GSXML.DOC_NODE_ELEM);
    338339                if (document_id != null)
    339340                {
    … …  
    353354            // should think about this more
    354355            // no structure request, so just put in a dummy doc node
    355             Element doc_node = this.doc.createElement(GSXML.DOC_NODE_ELEM);
     356            Element doc_node = doc.createElement(GSXML.DOC_NODE_ELEM);
    356357            if (document_id != null)
    357358            {
    … …  
    367368
    368369        // Build a request to obtain some document metadata
    369         Element dm_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
     370        Element dm_message = doc.createElement(GSXML.MESSAGE_ELEM);
    370371        String to = GSPath.appendLink(collection, "DocumentMetadataRetrieve"); // Hard-wired?
    371         Element dm_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
     372        Element dm_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
    372373        dm_message.appendChild(dm_request);
    373374        // Create a parameter list to specify the required metadata information
    … …  
    390391        }
    391392
    392         Element dm_param_list = createMetadataParamList(meta_names);
     393        Element dm_param_list = createMetadataParamList(doc,meta_names);
    393394        if (service_params != null)
    394395        {
    395             GSXML.addParametersToList(this.doc, dm_param_list, service_params);
     396            GSXML.addParametersToList(doc, dm_param_list, service_params);
    396397        }
    397398
    … …  
    399400
    400401        // create the doc node list for the metadata request
    401         Element dm_doc_list = this.doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
     402        Element dm_doc_list = doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
    402403        dm_request.appendChild(dm_doc_list);
    403404
    … …  
    410411
    411412            // Add the documentNode to the list
    412             Element dm_doc_node = this.doc.createElement(GSXML.DOC_NODE_ELEM);
     413            Element dm_doc_node = doc.createElement(GSXML.DOC_NODE_ELEM);
    413414            dm_doc_list.appendChild(dm_doc_node);
    414415            dm_doc_node.setAttribute(GSXML.NODE_ID_ATT, doc_node_id);
    … …  
    418419        // we also want a metadata request to the top level document to get
    419420        // assocfilepath - this could be cached too
    420         Element doc_meta_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
     421        Element doc_meta_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
    421422        dm_message.appendChild(doc_meta_request);
    422         Element doc_meta_param_list = this.doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
     423        Element doc_meta_param_list = doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
    423424        if (service_params != null)
    424425        {
    425             GSXML.addParametersToList(this.doc, doc_meta_param_list, service_params);
     426            GSXML.addParametersToList(doc, doc_meta_param_list, service_params);
    426427        }
    427428
    428429        doc_meta_request.appendChild(doc_meta_param_list);
    429         Element doc_param = this.doc.createElement(GSXML.PARAM_ELEM);
     430        Element doc_param = doc.createElement(GSXML.PARAM_ELEM);
    430431        doc_meta_param_list.appendChild(doc_param);
    431432        doc_param.setAttribute(GSXML.NAME_ATT, "metadata");
    … …  
    433434
    434435        // create the doc node list for the metadata request
    435         Element doc_list = this.doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
     436        Element doc_list = doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
    436437        doc_meta_request.appendChild(doc_list);
    437438
    438         Element doc_node = this.doc.createElement(GSXML.DOC_NODE_ELEM);
     439        Element doc_node = doc.createElement(GSXML.DOC_NODE_ELEM);
    439440        // the node we want is the root document node
    440441        if (document_id != null)
    … …  
    472473
    473474        // Build a request to obtain some document content
    474         Element dc_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
     475        Element dc_message = doc.createElement(GSXML.MESSAGE_ELEM);
    475476        to = GSPath.appendLink(collection, "DocumentContentRetrieve"); // Hard-wired?
    476         Element dc_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
     477        Element dc_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
    477478        dc_message.appendChild(dc_request);
    478479
    479480        // Create a parameter list to specify the request parameters - empty for now
    480         Element dc_param_list = this.doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
     481        Element dc_param_list = doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
    481482        if (service_params != null)
    482483        {
    483             GSXML.addParametersToList(this.doc, dc_param_list, service_params);
     484            GSXML.addParametersToList(doc, dc_param_list, service_params);
    484485        }
    485486
    … …  
    518519                        content = highlightQueryTerms(request, (Element) content);
    519520                    }
    520                     doc_nodes.item(i).appendChild(this.doc.importNode(content, true));
     521                    doc_nodes.item(i).appendChild(doc.importNode(content, true));
    521522                }
    522523                //GSXML.mergeMetadataLists(doc_nodes.item(i), dm_response_docs.item(i));
    … …  
    561562                    String enrich_service = (String) params.get(GSParams.SERVICE);
    562563                    // send a message to the service
    563                     Element enrich_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
    564                     Element enrich_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_PROCESS, enrich_service, userContext);
     564                    Element enrich_message = doc.createElement(GSXML.MESSAGE_ELEM);
     565                    Element enrich_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_PROCESS, enrich_service, userContext);
    565566                    enrich_message.appendChild(enrich_request);
    566567                    // check for parameters
    … …  
    568569                    if (e_service_params != null)
    569570                    {
    570                         Element enrich_pl = this.doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
    571                         GSXML.addParametersToList(this.doc, enrich_pl, e_service_params);
     571                        Element enrich_pl = doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
     572                        GSXML.addParametersToList(doc, enrich_pl, e_service_params);
    572573                        enrich_request.appendChild(enrich_pl);
    573574                    }
    574                     Element e_doc_list = this.doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
     575                    Element e_doc_list = doc.createElement(GSXML.DOC_NODE_ELEM + GSXML.LIST_MODIFIER);
    575576                    enrich_request.appendChild(e_doc_list);
    576                     e_doc_list.appendChild(this.doc.importNode(dc_response_doc, true));
     577                    e_doc_list.appendChild(doc.importNode(dc_response_doc, true));
    577578
    578579                    Node enrich_response = this.mr.process(enrich_message);
    … …  
    595596
    596597                dummy_node.setAttribute(GSXML.NODE_ID_ATT, modified_doc_id);
    597                 dummy_node.appendChild(this.doc.importNode(dc_response_doc_content, true));
     598                dummy_node.appendChild(doc.importNode(dc_response_doc_content, true));
    598599                // hack for simple type
    599600                if (document_type.equals(GSXML.DOC_TYPE_SIMPLE))
    … …  
    630631                    if (dn_id.equals(modified_doc_id))
    631632                    {
    632                         dn.appendChild(this.doc.importNode(dc_response_doc_content, true));
     633                        dn.appendChild(doc.importNode(dc_response_doc_content, true));
    633634                        break;
    634635                    }
    … …  
    663664    protected boolean getBackgroundData(Element page_response, String collection, UserContext userContext)
    664665    {
    665 
     666        Document doc = page_response.getOwnerDocument();
     667       
    666668        // create a message to process - contains requests for the collection
    667669        // description, the format element, the enrich services on offer
    668670        // these could all be cached
    669         Element info_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
     671        Element info_message = doc.createElement(GSXML.MESSAGE_ELEM);
    670672        String path = GSPath.appendLink(collection, "DocumentContentRetrieve");
    671673        // the format request - ignore for now, where does this request go to??
    672         Element format_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_FORMAT, path, userContext);
     674        Element format_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_FORMAT, path, userContext);
    673675        info_message.appendChild(format_request);
    674676
    … …  
    677679        if (provide_annotations)
    678680        {
    679             Element enrich_services_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_DESCRIBE, "", userContext);
     681            Element enrich_services_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_DESCRIBE, "", userContext);
    680682            enrich_services_request.setAttribute(GSXML.INFO_ATT, "serviceList");
    681683            info_message.appendChild(enrich_services_request);
    … …  
    699701            // set the format type
    700702            format_elem.setAttribute(GSXML.TYPE_ATT, "display");
    701             page_response.appendChild(this.doc.importNode(format_elem, true));
     703            page_response.appendChild(doc.importNode(format_elem, true));
    702704        }
    703705
    … …  
    707709
    708710            // a new message for the mr
    709             Element enrich_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
     711            Element enrich_message = doc.createElement(GSXML.MESSAGE_ELEM);
    710712            NodeList e_services = services_resp.getElementsByTagName(GSXML.SERVICE_ELEM);
    711713            boolean service_found = false;
    … …  
    714716                if (((Element) e_services.item(j)).getAttribute(GSXML.TYPE_ATT).equals("enrich"))
    715717                {
    716                     Element s = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_DESCRIBE, ((Element) e_services.item(j)).getAttribute(GSXML.NAME_ATT), userContext);
     718                    Element s = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_DESCRIBE, ((Element) e_services.item(j)).getAttribute(GSXML.NAME_ATT), userContext);
    717719                    enrich_message.appendChild(s);
    718720                    service_found = true;
    … …  
    724726
    725727                NodeList e_responses = enrich_response.getElementsByTagName(GSXML.RESPONSE_ELEM);
    726                 Element service_list = this.doc.createElement(GSXML.SERVICE_ELEM + GSXML.LIST_MODIFIER);
     728                Element service_list = doc.createElement(GSXML.SERVICE_ELEM + GSXML.LIST_MODIFIER);
    727729                for (int i = 0; i < e_responses.getLength(); i++)
    728730                {
    729731                    Element e_resp = (Element) e_responses.item(i);
    730                     Element e_service = (Element) this.doc.importNode(GSXML.getChildByTagName(e_resp, GSXML.SERVICE_ELEM), true);
     732                    Element e_service = (Element) doc.importNode(GSXML.getChildByTagName(e_resp, GSXML.SERVICE_ELEM), true);
    731733                    e_service.setAttribute(GSXML.NAME_ATT, e_resp.getAttribute(GSXML.FROM_ATT));
    732734                    service_list.appendChild(e_service);
    … …  
    741743    protected String getDocumentType(Element basic_doc_list, String collection, UserContext userContext, Element page_response)
    742744    {
    743         Element ds_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
     745        Document doc = basic_doc_list.getOwnerDocument();
     746       
     747        Element ds_message = doc.createElement(GSXML.MESSAGE_ELEM);
    744748        String to = GSPath.appendLink(collection, "DocumentStructureRetrieve");// Hard-wired?
    745         Element ds_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
     749        Element ds_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
    746750        ds_message.appendChild(ds_request);
    747751
    748752        // Create a parameter list to specify the required structure information
    749         Element ds_param_list = this.doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
    750         Element ds_param = this.doc.createElement(GSXML.PARAM_ELEM);
     753        Element ds_param_list = doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
     754        Element ds_param = doc.createElement(GSXML.PARAM_ELEM);
    751755        ds_param_list.appendChild(ds_param);
    752756        ds_param.setAttribute(GSXML.NAME_ATT, "info");
    … …  
    787791    protected Element highlightQueryTerms(Element request, Element dc_response_doc_content)
    788792    {
     793        Document doc = request.getOwnerDocument();
     794       
    789795        // do the query again to get term info
    790796        Element cgi_param_list = (Element) GSXML.getChildByTagName(request, GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
    … …  
    806812        String to = GSPath.appendLink(collection, service_name);
    807813
    808         Element mr_query_message = this.doc.createElement(GSXML.MESSAGE_ELEM);
    809         Element mr_query_request = GSXML.createBasicRequest(this.doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
     814        Element mr_query_message = doc.createElement(GSXML.MESSAGE_ELEM);
     815        Element mr_query_request = GSXML.createBasicRequest(doc, GSXML.REQUEST_TYPE_PROCESS, to, userContext);
    810816        mr_query_message.appendChild(mr_query_request);
    811817
    … …  
    813819        HashMap service_params = (HashMap) params.get("s1");
    814820
    815         Element query_param_list = this.doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
    816         GSXML.addParametersToList(this.doc, query_param_list, service_params);
     821        Element query_param_list = doc.createElement(GSXML.PARAM_ELEM + GSXML.LIST_MODIFIER);
     822        GSXML.addParametersToList(doc, query_param_list, service_params);
    817823        mr_query_request.appendChild(query_param_list);
    818824
    … …  
    969975        }
    970976
    971         return highlightQueryTermsInternal(content, query_term_variants, phrase_query_term_variants_hierarchy);
     977        return highlightQueryTermsInternal(doc, content, query_term_variants, phrase_query_term_variants_hierarchy);
    972978    }
    973979
    … …  
    975981     * Highlights query terms in a piece of text.
    976982     */
    977     private Element highlightQueryTermsInternal(String content, HashSet<String> query_term_variants, ArrayList<ArrayList<HashSet<String>>> phrase_query_term_variants_hierarchy)
     983    private Element highlightQueryTermsInternal(Document doc, String content, HashSet<String> query_term_variants, ArrayList<ArrayList<HashSet<String>>> phrase_query_term_variants_hierarchy)
    978984    {
    979985        // Convert the content string to an array of characters for speed
    … …  
    11221128
    11231129        // Now add the annotation tags into the document at the correct points
    1124         Element content_element = this.doc.createElement(GSXML.NODE_CONTENT_ELEM);
     1130        Element content_element = doc.createElement(GSXML.NODE_CONTENT_ELEM);
    11251131
    11261132        int last_wrote = 0;
    … …  
    11341140            {
    11351141                String preceding_text = new String(content_characters, last_wrote, (highlight_start - last_wrote));
    1136                 content_element.appendChild(this.doc.createTextNode(preceding_text));
     1142                content_element.appendChild(doc.createTextNode(preceding_text));
    11371143            }
    11381144
    … …  
    11411147            {
    11421148                String highlight_text = new String(content_characters, highlight_start, (highlight_end - highlight_start));
    1143                 Element annotation_element = GSXML.createTextElement(this.doc, "annotation", highlight_text);
     1149                Element annotation_element = GSXML.createTextElement(doc, "annotation", highlight_text);
    11441150                annotation_element.setAttribute("type", "query_term");
    11451151                content_element.appendChild(annotation_element);
    … …  
    11521158        {
    11531159            String remaining_text = new String(content_characters, last_wrote, (content_characters.length - last_wrote));
    1154             content_element.appendChild(this.doc.createTextNode(remaining_text));
     1160            content_element.appendChild(doc.createTextNode(remaining_text));
    11551161        }
    11561162
Note: See TracChangeset for help on using the changeset viewer.