joshdata/dc-code-editor

Prototype tool for editing the DC Code.

dc-code-editor's Issues

add/delete files

i.e. insert/remove a section

previews of pages with x:include tags don't work

Move/Delete

From time to time, it is necessary to move parts of the Code. For example, if you are breaking a Part into Subparts, you will need to move all of the sections in the Part into one or more of the Subparts. Right now, you can add the Subpart and add the files individually, but there should be any easier to manage multiple sections, and explicitly to move or delete individual levels.

Allow patches to be merged (squashed together) so that an existing patch can be revised.

Export the Code

Choose a patch and then export the code as of that patch to the code repository.

jot from npm?

jot is currently included via gitmodules - ideally this is just until it stabilizes and can be used via npm? For mid-development, you can use npm link for the dev workflow.

Feature: Showing what's already been changed

When you work on a patch, would be nice to include a list of files affected by the patch.

modify many sections at once

e.g. add something like "This section was amended by ___." to many files simultaneously

localCompare Error

Not sure what causes this, but I receive this error from time to time.

/home/user-data/dc-code-editor/editor/patches.js:422
                return a.localeCompare(b);
                         ^
TypeError: Cannot call method 'localeCompare' of undefined
    at cmp (/home/user-data/dc-code-editor/editor/patches.js:422:12)
    at /home/user-data/dc-code-editor/editor/patches.js:434:12
    at Array.sort (native)
    at Object.exports.sort_paths (/home/user-data/dc-code-editor/editor/patches.js:425:12)
    at /home/user-data/dc-code-editor/editor/views.js:43:15
    at /home/user-data/dc-code-editor/editor/patches.js:412:5
    at /home/user-data/dc-code-editor/editor/patches.js:412:5
    at /home/user-data/dc-code-editor/editor/repository.js:82:3
    at ChildProcess.<anonymous> (/home/user-data/dc-code-editor/editor/repository.js:31:3)
    at ChildProcess.EventEmitter.emit (events.js:98:17)

Encourage the user to commit & push their patch workspace repository.

A new error

Don't know exactly what caused this, but here's the error message:

git /home/user-data/dc-code-editor/base_code/ [ 'ls-tree',
  '-lz',
  '644eb4459db790849c1e607967bfb50ba18ba6fa',
  'Title-2/Chapter-12/Subchapter-IX/Part-B/Subpart-2/' ] null

/home/user-data/dc-code-editor/editor/views.js:322
.write(JSON.stringify( { "error": "Invalid XML in included file " + elem.get("
                                                                    ^
ReferenceError: elem is not defined
    at /home/user-data/dc-code-editor/editor/views.js:322:77
    at /home/user-data/dc-code-editor/node_modules/async/lib/async.js:232:13
    at /home/user-data/dc-code-editor/node_modules/async/lib/async.js:113:21
    at /home/user-data/dc-code-editor/node_modules/async/lib/async.js:24:16
    at /home/user-data/dc-code-editor/node_modules/async/lib/async.js:229:17
    at /home/user-data/dc-code-editor/editor/views.js:315:7
    at /home/user-data/dc-code-editor/editor/patches.js:553:5
    at /home/user-data/dc-code-editor/editor/patches.js:553:5
    at /home/user-data/dc-code-editor/editor/patches.js:553:5
    at /home/user-data/dc-code-editor/editor/patches.js:553:5
error: Forever detected script exited with code: 8

Patch Tree

• Show patches in a tree.
• Allow patches to be moved around like a git rebase.
• Give every patch an "effective date" field and encourage the user to put patches in order of effective date.

fast way to find a file by title/section

This is actually not trivial because the code repo is so large that listing every file in it is very slow.

Tests

There are testable parts of this code, like https://github.com/JoshData/dc-code-editor/blob/master/editor/repository.js (unless we switch to something like nodegit to avoid shelling out). Should make some tests with node-tap