Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Author vs Last Modified #79

Open
MrK- opened this Issue · 0 comments

1 participant

@MrK-

Since commit 7c312d110a2078987d9018032500d2b45526d292 Dradis displays the last modified time, and the author.

However, the author is not updated when a note is edited so what is displayed isn't strictly true now.

One proposed fix is to change the author field on such an event through modifying the notes browser code.
(assets/javascripts/dx/dradis.notes.NotesBrowserPanel.js)

    this.editor.on('save', function(newValue){
      if (this.note === undefined) {
        this.fields.grid.addNote(this.selectedNode, newValue);
        this.fields.preview.clear();
      } else {
        this.note.set('text', newValue);
+       this.note.set('author', dradis.author);
        this.fields.preview.update(newValue);
      }

However, in the long term perhaps a split between 'author' and 'last modified by' would make sense?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.