diff --git a/src/diff-parser.js b/src/diff-parser.js index bbe3c92..43fb396 100644 --- a/src/diff-parser.js +++ b/src/diff-parser.js @@ -64,12 +64,12 @@ var values; if (values = /^@@ -(\d+),\d+ \+(\d+),\d+ @@.*/.exec(line)) { - currentFile.isTripleDiff = false; + currentFile.isCombined = false; } else if (values = /^@@@ -(\d+),\d+ -\d+,\d+ \+(\d+),\d+ @@@.*/.exec(line)) { - currentFile.isTripleDiff = true; + currentFile.isCombined = true; } else { values = [0, 0]; - currentFile.isTripleDiff = false; + currentFile.isCombined = false; } oldLine = values[1]; @@ -120,23 +120,82 @@ // Unmerged paths, and possibly other non-diffable files // https://github.com/scottgonzalez/pretty-diff/issues/11 // Also, remove some useless lines - if (!line || utils.startsWith(line, "*") || - utils.startsWith(line, "new") || utils.startsWith(line, "index")) { + if (!line || utils.startsWith(line, "*")) { + //|| utils.startsWith(line, "new") || utils.startsWith(line, "index") return; } + /* Diff */ + var oldMode = /old mode (\d{6})/; + var newMode = /new mode (\d{6})/; + var deletedFileMode = /deleted file mode (\d{6})/; + var newFileMode = /new file mode (\d{6})/; + + var copyFrom = /copy from (.+)/; + var copyTo = /copy to (.+)/; + + var renameFrom = /rename from (.+)/; + var renameTo = /rename to (.+)/; + + var similarityIndex = /similarity index (\d+)%/; + var dissimilarityIndex = /dissimilarity index (\d+)%/; + var index = /index ([0-9a-z]+)..([0-9a-z]+) (\d{6})?/; + + /* Combined Diff */ + var combinedIndex = /index ([0-9a-z]+),([0-9a-z]+)..([0-9a-z]+)/; + var combinedMode = /mode (\d{6}),(\d{6})..(\d{6})/; + var combinedNewFile = /new file mode (\d{6})/; + var combinedDeletedFile = /deleted file mode (\d{6}),(\d{6})/; + var values = []; if (utils.startsWith(line, "diff")) { startFile(); } else if (currentFile && !currentFile.oldName && (values = /^--- a\/(\S+).*$/.exec(line))) { currentFile.oldName = values[1]; + currentFile.language = getExtension(currentFile.oldName, currentFile.language); } else if (currentFile && !currentFile.newName && (values = /^\+\+\+ [b]?\/(\S+).*$/.exec(line))) { currentFile.newName = values[1]; - - var fileSplit = currentFile.newName.split("."); - currentFile.language = fileSplit[fileSplit.length - 1]; + currentFile.language = getExtension(currentFile.newName, currentFile.language); } else if (currentFile && utils.startsWith(line, "@@")) { startBlock(line); + } else if ((values = oldMode.exec(line))) { + currentFile.oldMode = values[1]; + } else if ((values = newMode.exec(line))) { + currentFile.newMode = values[1]; + } else if ((values = deletedFileMode.exec(line))) { + currentFile.deletedFileMode = values[1]; + } else if ((values = newFileMode.exec(line))) { + currentFile.newFileMode = values[1]; + } else if ((values = copyFrom.exec(line))) { + currentFile.oldName = values[1]; + currentFile.isCopy = true; + } else if ((values = copyTo.exec(line))) { + currentFile.newName = values[1]; + currentFile.isCopy = true; + } else if ((values = renameFrom.exec(line))) { + currentFile.oldName = values[1]; + currentFile.isRename = true; + } else if ((values = renameTo.exec(line))) { + currentFile.newName = values[1]; + currentFile.isRename = true; + } else if ((values = similarityIndex.exec(line))) { + currentFile.unchangedPercentage = values[1]; + } else if ((values = dissimilarityIndex.exec(line))) { + currentFile.changedPercentage = values[1]; + } else if ((values = index.exec(line))) { + currentFile.checksumBefore = values[1]; + currentFile.checksumAfter = values[2]; + values[2] && (currentFile.mode = values[3]); + } else if ((values = combinedIndex.exec(line))) { + currentFile.checksumBefore = [values[2], values[3]]; + currentFile.checksumAfter = values[1]; + } else if ((values = combinedMode.exec(line))) { + currentFile.oldMode = [values[2], values[3]]; + currentFile.newMode = values[1]; + } else if ((values = combinedNewFile.exec(line))) { + currentFile.newFileMode = values[1]; + } else if ((values = combinedDeletedFile.exec(line))) { + currentFile.deletedFileMode = values[1]; } else if (currentBlock) { createLine(line); } @@ -148,6 +207,12 @@ return files; }; + function getExtension(filename, language) { + var nameSplit = filename.split("."); + if (nameSplit.length > 1) return nameSplit[nameSplit.length - 1]; + else return language; + } + if (typeof module !== 'undefined' && module.exports) { module.exports.DiffParser = new DiffParser(); } else if (typeof global.DiffParser === 'undefined') { diff --git a/src/diff2html.js b/src/diff2html.js index ad5664f..5d769d3 100644 --- a/src/diff2html.js +++ b/src/diff2html.js @@ -30,7 +30,6 @@ */ Diff2Html.prototype.getPrettyHtmlFromDiff = function (diffInput, config) { var diffJson = diffParser.generateDiffJson(diffInput); - var configOrEmpty = config || {}; return htmlPrinter.generateLineByLineJsonHtml(diffJson, configOrEmpty); }; diff --git a/src/line-by-line-printer.js b/src/line-by-line-printer.js index 20b040d..578662b 100644 --- a/src/line-by-line-printer.js +++ b/src/line-by-line-printer.js @@ -17,19 +17,24 @@ LineByLinePrinter.prototype.generateLineByLineJsonHtml = function (diffFiles, config) { return "