discourse/discourse — pull request #1477 by velesin (2013-09-27 18:04:05+00:00)
Title: removes unused builders param from Discourse.Model.mergeAttributes

Full text search over the whole project tree reveals that builders param is never used. Considering that Discourse.Model class was last modified about 4 months ago I assume this is some forgotten, legacy code and can be removed.

WARNING: I'm aware that the test suite for the JS part of the app is very incomplete and I'm too new to the project to be able to manually discover and test all the edge cases - so in theory there may be a situation that mergeAttributes method is invoked somewhere in a way difficult to catch by a full text search (e.g. method name built dynamically by string concatenation) - so please verify that my commit doesn't break anything before accepting this pull request.