Is it possible to customize the id field? about crud HOT 12 CLOSED

ok, this just worked for me in the entity level:

  @PrimaryColumn({
    name: "bt_brand_id",
    length: 20,
    type: "varchar",
  })
  public id!: string;

But if there's a way on the that lib level, i think i will prefer it

from crud.

Shipped with v3.0.0

from crud.

I think it's possible. We already have params in CrudOptions object, where you can map id path param to an entity's field. But some adjustment in RepositoryService is needed in that case. I'll try to do this ASAP. Cheers!

from crud.

Valid point by @dhaspden
@nestjsx/crud assumes the id column to be a number in both the CrudController.getOneBase and RepositoryService.getOne

We rarely use the type number as an ID type of entities which are available to public to avoid making the total number of records in our database easy to discover.

from crud.

I took a look briefly at this since I was also interested in having this. I think it might be a bit more nuanced because a lot of places assume number type for the identifier field (CrudController and the decorators which setup parsing the parameter to a number.)

I was trying to think about how we could use the params in CrudOptions and figured it might be unintuitive to map them somehow to the service since you could potentially have more than one attribute in that params option. I wonder if maybe a new RestfulOptions, maybe like idColumnName would make more sense.

This leads me to another nuance which is handling the case where this id column isn't a number but instead something like a UUID. We would need a way to specify the type of that identifier field. I'm no expert on Typescript (yet) but perhaps that could be solved with yet another option, say idColumnType? I'm not sure if it'd work but you could also make the service a two value generic where T is the entity type and U is the type of the key?

Sorry about puking my thoughts out on the page but hopefully that gets through. I'd be really interested in seeing this happen since I like to have uuid columns for my public facing API to avoid having to disclose internal database ids.

Edit

If you set the params object to be

{
  id: 'uuid'
}

For example, would the library process that correctly? If so I guess we could technically parse the identifier something along the lines of

const idFieldName = params && params.id ? params.id : 'id';

D

from crud.

+1: id should be type string as option (UUID)

from crud.

change to createParamMetadata(RouteParamtypes.PARAM, 0, [], 'id'),

from crud.

Thank you!

from crud.

+1: id should be type string as option (UUID)

from crud.

Thank you very much

from crud.

Ok,
I'm not sure how to use it XD
I want the name of the id field to be brand_id

from crud.

I can now do just this:

@PrimaryGeneratedColumn({
        type:"integer",
        name:"population_id"
        })
    population_id:number;

and it's working-

from crud.