santakadev / aixada Goto Github PK

Refactor create user/member use case:

• Move stored procedure to domain/application layer
• Make code testable and follow SOLID principles

Possible improvements

• Add backend validation
• Security

In new_user_member stored procedure there is a limit of 1000 in ID generation. I don't know if this is here for some reason.

  	select 
  		max(id)+1 
  	into 
  		the_user_id
  	from 
  		aixada_user 
  	where 
  		id<1000;

get_param can transform retrieved data but it seems that transformation it is no used anywhere.

There is some type of symlinks to jQuery and jQuery UI versions:

js/jquery/jquery.js
js/jqueryui/jqueryui,js
css/ui-themes/smoothness/jqueryui.css

When running Aixada in a Windows host through Docker, this links are not resolved.

In aixada_user table member_id column seams redundant. The new_user_member stored procedure assigns the same ID for user and member, so is not possible to have a ID mismatch if all user and member creation goes through this stored procedure.