Preserve key index when re-adding an existing key

James Moger

2014-03-27 ab07d0d86d8b64b4f7c88b45bc81f1eec22105db

 src/main/java/com/gitblit/wicket/MarkupProcessor.java

@@ -15,6 +15,8 @@
 */

package com.gitblit.wicket;



import static org.pegdown.FastEncoder.encode;



import java.io.Serializable;

import java.io.StringWriter;

import java.io.UnsupportedEncodingException;

@@ -22,6 +24,7 @@
import java.text.MessageFormat;

import java.util.ArrayList;

import java.util.Arrays;

import java.util.Collections;

import java.util.HashMap;

import java.util.List;

import java.util.Map;

@@ -39,8 +42,14 @@
import org.eclipse.mylyn.wikitext.textile.core.TextileLanguage;

import org.eclipse.mylyn.wikitext.tracwiki.core.TracWikiLanguage;

import org.eclipse.mylyn.wikitext.twiki.core.TWikiLanguage;

import org.pegdown.DefaultVerbatimSerializer;

import org.pegdown.LinkRenderer;

import org.pegdown.ToHtmlSerializer;

import org.pegdown.VerbatimSerializer;

import org.pegdown.ast.ExpImageNode;

import org.pegdown.ast.RefImageNode;

import org.pegdown.ast.WikiLinkNode;

import org.pegdown.plugins.ToHtmlSerializerPlugin;

import org.slf4j.Logger;

import org.slf4j.LoggerFactory;



@@ -52,6 +61,7 @@
import com.gitblit.utils.StringUtils;

import com.gitblit.wicket.pages.DocPage;

import com.gitblit.wicket.pages.RawPage;

import com.google.common.base.Joiner;



/**

 * Processes markup content and generates html with repository-relative page and

@@ -296,6 +306,34 @@
    */

   private void parse(final MarkupDocument doc, final String repositoryName, final String commitId) {

      LinkRenderer renderer = new LinkRenderer() {



         @Override

         public Rendering render(ExpImageNode node, String text) {

            if (node.url.indexOf("://") == -1) {

               // repository-relative image link

               String path = doc.getRelativePath(node.url);

               String url = getWicketUrl(RawPage.class, repositoryName, commitId, path);

               return new Rendering(url, text);

            }

            // absolute image link

            return new Rendering(node.url, text);

         }



         @Override

         public Rendering render(RefImageNode node, String url, String title, String alt) {

            Rendering rendering;

            if (url.indexOf("://") == -1) {

               // repository-relative image link

               String path = doc.getRelativePath(url);

               String wurl = getWicketUrl(RawPage.class, repositoryName, commitId, path);

               rendering = new Rendering(wurl, alt);

            } else {

               // absolute image link

               rendering = new Rendering(url, alt);

            }

            return StringUtils.isEmpty(title) ? rendering : rendering.withAttribute("title", encode(title));

         }



         @Override

         public Rendering render(WikiLinkNode node) {

            String path = doc.getRelativePath(node.getText());

@@ -358,7 +396,59 @@
      }



      String getRelativePath(String ref) {

         return ref.charAt(0) == '/' ? ref.substring(1) : (getCurrentPath() + ref);

         if (ref.charAt(0) == '/') {

            // absolute path in repository

            return ref.substring(1);

         } else {

            // resolve relative repository path

            String cp = getCurrentPath();

            if (StringUtils.isEmpty(cp)) {

               return ref;

            }

            // this is a simple relative path resolver

            List<String> currPathStrings = new ArrayList<String>(Arrays.asList(cp.split("/")));

            String file = ref;

            while (file.startsWith("../")) {

               // strip ../ from the file reference

               // drop the last path element

               file = file.substring(3);

               currPathStrings.remove(currPathStrings.size() - 1);

            }

            currPathStrings.add(file);

            String path = Joiner.on("/").join(currPathStrings);

            return path;

         }

      }

   }



   /**

    * This class implements a workaround for a bug reported in issue-379.

    * The bug was introduced by my own pegdown pull request #115.

    *

    * @author James Moger

    *

    */

   public static class WorkaroundHtmlSerializer extends ToHtmlSerializer {



       public WorkaroundHtmlSerializer(final LinkRenderer linkRenderer) {

          super(linkRenderer,

                Collections.<String, VerbatimSerializer>singletonMap(VerbatimSerializer.DEFAULT, DefaultVerbatimSerializer.INSTANCE),

                Collections.<ToHtmlSerializerPlugin>emptyList());

          }

       private void printAttribute(String name, String value) {

           printer.print(' ').print(name).print('=').print('"').print(value).print('"');

       }



       /* Reimplement print image tag to eliminate a trailing double-quote */

      @Override

       protected void printImageTag(LinkRenderer.Rendering rendering) {

           printer.print("<img");

           printAttribute("src", rendering.href);

           printAttribute("alt", rendering.text);

           for (LinkRenderer.Attribute attr : rendering.attributes) {

               printAttribute(attr.name, attr.value);

           }

           printer.print("/>");

       }

   }

}