Quality size settings save now, with fancy messages

NzbDrone.Api/Qualities/QualitySizeResource.cs

@@ -5,6 +5,7 @@ namespace NzbDrone.Api.Qualities
 {
     public class QualitySizeResource : RestResource
     {
+        public Int32 QualityId { get; set; }
         public String Name { get; set; }
         public Int32 MinSize { get; set; }
         public Int32 MaxSize { get; set; }

UI/JsLibraries/backbone.mutators.deep.model.js

@@ -0,0 +1,211 @@
+/*! Backbone.Mutators - v0.4.0
+------------------------------
+Build @ 2013-05-01
+Documentation and Full License Available at:
+http://asciidisco.github.com/Backbone.Mutators/index.html
+git://github.com/asciidisco/Backbone.Mutators.git
+Copyright (c) 2013 Sebastian Golasch <public@asciidisco.com>
+
+Permission is hereby granted, free of charge, to any person obtaining a
+copy of this software and associated documentation files (the "Software"),
+to deal in the Software without restriction, including without limitation
+the rights to use, copy, modify, merge, publish, distribute, sublicense,
+and/or sell copies of the Software, and to permit persons to whom the
+
+Software is furnished to do so, subject to the following conditions:
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.
+IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM,
+DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
+ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
+IN THE SOFTWARE.*/
+(function (root, factory, undef) {
+    'use strict';
+
+    if (typeof exports === 'object') {
+        // Node. Does not work with strict CommonJS, but
+        // only CommonJS-like enviroments that support module.exports,
+        // like Node.
+        module.exports = factory(require('underscore'), require('backbone.deepmodel'));
+    } else if (typeof define === 'function' && define.amd) {
+        // AMD. Register as an anonymous module.
+        define(['underscore', 'backbone.deepmodel'], function (_, DeepModel) {
+            // Check if we use the AMD branch of Back
+            _ = _ === undef ? root._ : _;
+            DeepModel = DeepModel === undef ? root.DeepModel : DeepModel;
+            return (root.returnExportsGlobal = factory(_, DeepModel, root));
+        });
+    } else {
+        // Browser globals
+        root.returnExportsGlobal = factory(root._, root.DeepModel);
+    }
+
+// Usage:
+//
+// Note: This plugin is UMD compatible, you can use it in node, amd and vanilla js envs
+//
+// Vanilla JS:
+// <script src="underscore.js"></script>
+// <script src="backbone.js"></script>
+// <script src="backbone.mutators.js"></script>
+//
+// Node:
+// var _ = require('underscore');
+// var Backbone = require('backbone');
+// var Mutators = require('backbone.mutators');
+//
+//
+// AMD:
+// define(['underscore', 'backbone', 'backbone.mutators'], function (_, Backbone, Mutators) {
+//    // insert sample from below
+//    return User;
+// });
+//
+// var User = Backbone.Model.extend({
+//    mutators: {
+//        fullname: function () {
+//            return this.firstname + ' ' + this.lastname;
+//        }
+//    },
+//
+//    defaults: {
+//        firstname: 'Sebastian',
+//        lastname: 'Golasch'
+//    }
+// });
+//
+// var user = new User();
+// user.get('fullname') // returns 'Sebastian Golasch'
+// user.toJSON() // return '{firstname: 'Sebastian', lastname: 'Golasch', fullname: 'Sebastian Golasch'}'
+
+}(this, function (_, DeepModel, root, undef) {
+    'use strict';
+
+    // check if we use the amd branch of backbone and underscore
+    DeepModel = DeepModel === undef ? root.DeepModel : DeepModel;
+    _ = _ === undef ? root._ : _;
+
+    // extend backbones model prototype with the mutator functionality
+    var Mutator     = function () {},
+        oldGet      = DeepModel.DeepModel.prototype.get,
+        oldSet      = DeepModel.DeepModel.prototype.set,
+        oldToJson   = DeepModel.DeepModel.prototype.toJSON;
+
+    // This is necessary to ensure that Models declared without the mutators object do not throw and error
+    Mutator.prototype.mutators = {};
+
+    // override get functionality to fetch the mutator props
+    Mutator.prototype.get = function (attr) {
+        var isMutator = this.mutators !== undef;
+
+        // check if we have a getter mutation
+        if (isMutator === true && _.isFunction(this.mutators[attr]) === true) {
+            return this.mutators[attr].call(this);
+        }
+
+        // check if we have a deeper nested getter mutation
+        if (isMutator === true && _.isObject(this.mutators[attr]) === true && _.isFunction(this.mutators[attr].get) === true) {
+            return this.mutators[attr].get.call(this);
+        }
+
+        return oldGet.call(this, attr);
+    };
+
+    // override set functionality to set the mutator props
+    Mutator.prototype.set = function (key, value, options) {
+        var isMutator = this.mutators !== undef,
+            ret = null,
+            attrs = null;
+
+        // seamleassly stolen from backbone core
+        // check if the setter action is triggered
+        // using key <-> value or object
+        if (_.isObject(key) || key === null) {
+            attrs = key;
+            options = value;
+        } else {
+            attrs = {};
+            attrs[key] = value;
+        }
+
+        // check if we have a deeper nested setter mutation
+        if (isMutator === true && _.isObject(this.mutators[key]) === true) {
+
+            // check if we need to set a single value
+            if (_.isFunction(this.mutators[key].set) === true) {
+                ret = this.mutators[key].set.call(this, key, attrs[key], options, _.bind(oldSet, this));
+            } else if(_.isFunction(this.mutators[key])){
+                ret = this.mutators[key].call(this, key, attrs[key], options, _.bind(oldSet, this));
+            }
+        }
+
+        if (_.isObject(attrs)) {
+            _.each(attrs, _.bind(function (attr, attrKey) {
+                var cur_ret = null;
+                if (isMutator === true && _.isObject(this.mutators[attrKey]) === true) {
+                    // check if we need to set a single value
+
+                    var meth = this.mutators[attrKey];
+                    if(_.isFunction(meth.set)){
+                        meth = meth.set;
+                    }
+
+                    if(_.isFunction(meth)){
+                        if (options === undef || (_.isObject(options) === true && options.silent !== true && (options.mutators !== undef && options.mutators.silent !== true))) {
+                            this.trigger('mutators:set:' + attrKey);
+                        }
+                        cur_ret = meth.call(this, attrKey, attr, options, _.bind(oldSet, this));
+                    }
+
+                }
+                if (cur_ret === null) {
+                    cur_ret = _.bind(oldSet, this)(attrKey, attr, options);
+                }
+
+                if (ret !== false) { ret = cur_ret; }
+
+            }, this));
+        }
+
+        //validation purposes
+        if (ret !== null) {
+            return ret;
+        }
+
+        return oldSet.call(this, key, value, options);
+    };
+
+    // override toJSON functionality to serialize mutator properties
+    Mutator.prototype.toJSON = function () {
+        // fetch ye olde values
+        var attr = oldToJson.call(this);
+        // iterate over all mutators (if there are some)
+        _.each(this.mutators, _.bind(function (mutator, name) {
+            // check if we have some getter mutations
+            if (_.isObject(this.mutators[name]) === true && _.isFunction(this.mutators[name].get)) {
+                attr[name] = _.bind(this.mutators[name].get, this)();
+            } else {
+                attr[name] = _.bind(this.mutators[name], this)();
+            }
+        }, this));
+
+        return attr;
+    };
+
+    // override get functionality to get HTML-escaped the mutator props
+    Mutator.prototype.escape = function (attr){
+        var val = this.get(attr);
+        return _.escape(val == null ? '' : '' + val);
+    };
+
+    // extend the models prototype
+    _.extend(DeepModel.DeepModel.prototype, Mutator.prototype);
+
+    // make mutators globally available under the Backbone namespace
+    DeepModel.Mutators = Mutator;
+    return Mutator;
+}));

UI/Quality/QualitySizeModel.js

@@ -2,9 +2,21 @@
 
 define(
     [
-        'backbone'
+        'app',
-    ], function (Backbone) {
+        'Settings/SettingsModelBase'
-        return Backbone.Model.extend({
+    ], function (App, ModelBase) {
+        return ModelBase.extend({
+
+            baseInitialize: ModelBase.prototype.initialize,
+
+            initialize: function () {
+                var name = this.get('name');
+
+                this.successMessage = 'Saved ' + name + ' size settings';
+                this.errorMessage = 'Couldn\'t save ' + name + ' size settings';
+
+                this.baseInitialize.call(this);
+            },
 
             mutators: {
                 thirtyMinuteSize: function () {

UI/Settings/SettingsModelBase.js

@@ -6,8 +6,6 @@ define(['app',
     var model = DeepModel.DeepModel.extend({
 
         initialize: function () {
-
-           // App.vent.on(App.Commands.SaveSettings, this.saveSettings, this);
             this.listenTo(App.vent, App.Commands.SaveSettings, this.saveSettings);
         },
 

UI/app.js

@@ -10,6 +10,7 @@ require.config({
         'handlebars.helpers'  : 'JsLibraries/handlebars.helpers',
         'bootstrap'           : 'JsLibraries/bootstrap',
         'backbone.mutators'   : 'JsLibraries/backbone.mutators',
+        'backbone.mutators.deep.model': 'JsLibraries/backbone.mutators.deep.model',
         'backbone.deepmodel'  : 'JsLibraries/backbone.deep.model',
         'backbone.pageable'   : 'JsLibraries/backbone.pageable',
         'backbone.modelbinder': 'JsLibraries/backbone.modelbinder',
@@ -173,7 +174,8 @@ define(
 
         require(
             [
-                'libs/backbone.mutators'
+                'libs/backbone.mutators',
+                'libs/backbone.mutators.deep.model'
             ]);