Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#387 closed improve feature (fixed)

Not showing the picture of not public profile for not logged user

Reported by: jeanyves Owned by: lemon-head
Priority: major Milestone: 0.1.4
Component: unknown Keywords: publicprofile picture forum
Cc: lemon-head, micha

Description

I would like someone to make a safety precaution. When a not logged member sur the BW website, he must not be allowed to see the picture of a member who has not choose to make his profile public.

This is currently a problem in the forum. I think the forum call MOD_layoutbits::smallUserPic_username($post->user_handle) so I suggess the following : To add in MOD_layoutbits::smallUserPic_username a code to :

  • do as usual if the current user is logged on (and please do this with the IsLoggedIn or rox equivalent function)
  • elsewhere, if not LoggedIn?, test is the member has a public profile (this is done in BW with the IsPublic?($post->user_handle))

This should avoid discussion or complain with google showing picture in our forum for members who have not choose to have a public profile

Change History (10)

comment:1 Changed 11 years ago by lemon-head

  • follow_up changed from none to test
  • Owner set to lemon-head
  • Status changed from new to assigned

quick fix in [3950].

Later work should allow users to upload a separate avatar for non-logged viewers. Or allow to hide the profile from public, but still show the picture.

comment:2 Changed 11 years ago by lemon-head

See also [3952]. It adds pictures for this purpose.

Some more real artwork is needed!

comment:3 Changed 11 years ago by matrixpoint

  • follow_up changed from test to test on alpha
  • freq_reported set to 1
  • show_on_bw set to 0

Moved to alpha in r4100. Had to go as part of #347, #349.

comment:4 Changed 11 years ago by matrixpoint

  • follow_up changed from test on alpha to test

I decided to move this back to test, because jeanyves recommended going to production quickly with the changes that were made after these changes. So, I had to apply Micha's changes in r3984 on alpha without Andreas' changes in r3950, r3952.

comment:5 Changed 11 years ago by micha

  • follow_up changed from test to move to alpha

I don't get it. So, we still should move this to alpha, no?

comment:6 Changed 11 years ago by feuerdaemon

  • Milestone changed from unassigned to Bugs - not sheduled

comment:7 Changed 11 years ago by micha

  • follow_up changed from move to alpha to test on alpha

comment:8 Changed 11 years ago by micha

  • Milestone changed from Bugs - not sheduled to 0.1.4 - improving userinterface for members and volunteers and start work on big 0.2 tasks

moved to alpha in [4564]. Please test.

comment:9 Changed 11 years ago by philipp

  • Cc micha added
  • follow_up changed from test on alpha to release
  • Resolution set to fixed
  • Status changed from assigned to closed

works for me, release

comment:10 Changed 11 years ago by micha

  • follow_up changed from release to none

moved to production in r4597

Note: See TracTickets for help on using tickets.