Pass all method parameters in constructor

[tarmolov]

Implementation of the interface:

var myMethod = new ApiMethod({
    name: 'myMethod',
    params: {
        name: {type: 'String', required: true}
    },
    options: {
        hiddenOnDocPage: true
    },
    action: function (params) {
        // ...
    }
});

[coveralls]

Coverage remained the same when pulling eccd71c on method into 0b5678a on master.

[dodev]

So we're dropping support for the old declaration style? I don't think that's a good idea. Maybe we can support both? I know it's version 0.x, but that seems too radical.

[dodev]

Here the error should be something like 'Some of the required method properties were not specified (name, action).'

[tarmolov]

Firstly, I kept the old version but then pruned it.

Services should rewrite their code anyway. Renaming changes was radical, too.

Ok, I'll revert removing old methods, mark them as @deprecated, and keep them in the ApiMethod class. @lemmy- voted for keeping old methods, too.

[coveralls]

Coverage decreased (-1.56%) when pulling 800d94a on method into 0b5678a on master.

[coveralls]

Coverage remained the same when pulling 5b09986 on method into 0b5678a on master.

[tarmolov]

pr with updated normalize function waits for merging this pr. Please take a look, guys.

[watch-the-stars]

... with data ... or with params

[watch-the-stars]

LGTM

Upd: though, I'm not a guy ;)

[tarmolov]

(guys) амер. ; разг. ребята, люди (используется при обращении к лицам мужского и женского пола)

http://www.lingvo-online.ru/en/Translate/en-ru/guy

[coveralls]

Coverage remained the same when pulling 1142812 on method into 0b5678a on master.

[tarmolov]

I did what @dodev suggested. So, let's say that 1 LGTM is sufficient for this pr ;)