*Implemented CORE-1274,1278,1312

Hi Sheeban,

Changes made as you have suggested .


Thanks,

Cyril


On Sat, Jul 12, 2014 at 3:40 AM, sheebans <notifications@github.com> wrote:

> In
> api/src/main/java/org/ednovo/gooru/domain/service/ScollectionServiceImpl.java:
>
> >  		} else {
> >  			throw new NotFoundException(generateErrorMessage(GL0056, _COLLECTION));
> >  		}
> >  		this.redisService.bulkKeyDelete("v2-organize-data-" + user.getPartyUid() + "*");
> >
> >  	}
> > +	
> > +	public Boolean isOwnerOrContentAdmin(User resourceOwner, User user){
> > +		Boolean isUserPermission = false;
> > +		if(user != null && userService.isContentAdmin(user)){
> > +			isUserPermission = true;
>
> Can you name the method name has "hasUnrestrictedContentAccess" ? and
> instead of checking the owner of the content user, check in content
> permission table. Move this method to "OperationAuthorizer" class file
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Gooru/Gooru-Core-API/pull/370/files#r14846608>.
>



-- 
Thanks ,

Cyril M