This project is read-only.
1

Closed

Lame architecture

description

var context = new NetSqlAzManStorageDataContext(connection);
context.ApplicationGroupMemberDelete(groupMemberId, null); <-- ApplicationId is marked as Int? so i legaly can set null here
 
Then
 
[netsqlazman_ApplicationGroupMemberDelete]:
"...AND dbo.[netsqlazman_CheckApplicationPermissions](@ApplicationId, 2) = 1"
 
[netsqlazman_CheckApplicationPermissions]
"IF @ApplicationId IS NULL OR @ROLEID IS NULL SET @RESULT = 0"
 
And that crap everywhere. Man - you need to hire someone to make code review. I've never seen so shitty API as here.
Closed Jul 24, 2012 at 9:44 PM by aferende
Closed for user bad-language !

comments

aferende wrote Jul 24, 2012 at 9:43 PM

First of all I invite you to use a non-offensive language.
If you do not like the code, do not use the project.
Closed.

wrote Jul 24, 2012 at 9:44 PM

wrote Feb 22, 2013 at 12:33 AM

wrote May 16, 2013 at 12:00 PM