.all().one() does not work for numbers

This works:

> Restangular.all("models").one("1").getRestangularUrl()
/api/models/1

This also works:

> Restangular.one("models", 1).getRestangularUrl()
/api/models/1

But this triggers an error:

> Restangular.all("models").one(1).getRestangularUrl()
Error: You're creating a Restangular entity with the number instead of the route or the parent. You can't call .one(12)

Shouldn’t this be allowed for consistency reasons?

[pauldijou]

Sample 1 and 3 do not represent the same resource.

Restangular.all("models").one("1") means the sub-resource named 1 with currently no id inside the resource models (used to create an empty resource before doing a POST) while Restangular.all("models").one(1) would mean resource with id 1 inside collection models but that’s what Restangular.one("models", 1) is for, and nobody really wants to have two ways to do the same thing.

[diosney]

Same issue here, had to typecast to String in order to use it properly:

 if (instance_id) {
      return model
        .one(String(instance_id));
    }

+1 to @ntrrgc way of using it, like tasks.one(15).get(), and if a String, parse internally to number.