import $ from 'jquery';
import _ from 'underscore';
import Row from '../row/collection';
import Author from '../author/model';

export default Row.extend({
    model: Author,

    //Filters out non-array levels
    parse: function (response) {
        response = Row.prototype.parse.call(this, response);
        return response.Author;
    },

    //Converts authors to a list of pointers. It leverages Backbone's default ID (CID) as 
    //for the hash pointers. 
    //NOTE: By default, Backbone ignores models with existing IDs but will nevertheless 
    //return them as part of the add operation. If the authors are not in the collection's
    //hash, it also adds them.
    toPointers: function (response) {
        return _.pluck(this.add(response, {parse: true}), 'cid');
    },

    //Updates the article list for each of the authors of the present article.
    changeArticles: function (authorCIDs, articleCID) {
        authorCIDs.forEach((authorCID) => {
            this._byId[authorCID].changeArticlesAttr(articleCID);
        });
    },

    //Uses base64 as a quick and dirty form of unique ID for an article if not set.
    //NOTE: models are looked up in the collection to check if they already exist, 
    //before parsing. Hence why setting an ID at parsing time is too late.
    //TODO: Two authors can have the same name and initials. Find a safer and more efficient
    //alternative if available or make IDs a requirement for authors on the backend.
    modelId: function (attributes) {
        let id = Row.prototype.modelId.call(this, attributes);

        if (!id) {
            id = this.getBase64(attributes);
            attributes.id = id;
        }
        
        return id;
    },

    //Encodes a string to base64. Assuming every full name is unique, it uses the model's 
    //concatenated attributes as the input. 
    //SOURCE: https://developer.mozilla.org/en/docs/Web/API/WindowBase64/Base64_encoding_and_decoding
    getBase64: function (attributes) {
        const message = _.values(attributes).join();

        return btoa(encodeURIComponent(message).replace(/%([0-9A-F]{2})/g, function(match, p1) {
            return String.fromCharCode('0x' + p1);
        }));
    }
});

@@ -348,14 +348,14 @@
   //
-Conver
+Extrac
 ts a
@@ -363,106 +363,159 @@
 thor
-s to a list of pointers. It leverages Backbone's default ID (CID) as %0A    //for the hash pointers.
+ data from the article, converting the author list into a series of%0A    //pointers during the process. Each pointer is a CID, Backbone's generated ID. 
  %0A  

Improve language of comments