[$50] Don't apply "clearObject" method and remove it

[maxceem]

Currently, we are applying method "helper.clearObject" to all the responses.

This method:

• Transforms response = null to undefined (why?)
• If the response is an array, then it removes all the null values form the array (I guess this might hide some issue in the logic which creates this array)
• If response is object and has fields with null or [] fields, it removes such fields (actually null means that we have the field but without value, while no field, means field doesn't exists. Also, returning [] makes it easier on the client side, as we can count that value is array and we can work with it like array)

In general this method feels like make the responses less unified.

So I don't see benefits from such clearing, while see some drawbacks and suggest to remove this method, and don't apply it to responses.

We have to make sure that the code didn't got broken after removing it.

If someone knows any benefits of this method, please, let me know.

[maxceem]

Challenge https://www.topcoder.com/challenges/f4a9d5e1-aa85-4be8-9fc2-085baedd2fa3 has been created for this ticket.

This is an automated message for maxceem via Topcoder X

[maxceem]

@imcaizheng this is ready for pickup.

The fix by itself is trivial. But please, verify that this fix doesn't break any of the affected requests (including team requests). If you see any drawbacks, please, let me know.

[maxceem]

Challenge https://www.topcoder.com/challenges/f4a9d5e1-aa85-4be8-9fc2-085baedd2fa3 has been assigned to aaron2017.

This is an automated message for maxceem via Topcoder X

[imcaizheng]

PR created #135