Feedback for MembersList Enhanced

peregrine

continued from here
http://vanillaforums.org/discussion/21173/a-new-enhanced-member-list-plugin

if it works in your version let me know what version of this plugin and what version of vanilla.

peregrine

If you are using version 1.0 or 1.1,

@pamela and @sheila (you asked about these features a while ago).

I suggest you use Members List enhanced v.2.0

I added a whole bunch of settings features in the dashboard for selection of what columns to view, number of users to list per page, also

there are two permissions settings in

->Roles and Permissions ->edit role

toward the bottom of roles and permissions

you will see a group
=====GenView======

 
Plugins              GenView    IPEmailView     
MembersListEnh   x


click the GenView for any users you want to view (email and ip addresses WILL NOT be DISPLAYED even if you may have picked email and ip address to display as columns)        

===== IPEmailView =====

Plugins              GenView    IPEmailView     
MembersListEnh                          x


click the IPEmailView for any users you want to view every memberlist column you selected (if you selected picked email and ip address to display as columns the WILL be displayed in the table)

GenView is kind of a restricted view

IPEmailView os more of an Administrative view

Pamela

Many thanks for your suggestion it's approved now... friendly dashboard settings way to select columns, is perfect

Pamela

an idea, not more add a side panel to select which members role is displayed !

peregrine

Pamela said:
an idea, not more add a side panel to select which members role is displayed !

perhaps...

Pamela

Really not requested but may be, we could add sorting to roles column in the future is not ?

Pamela

Recently, a new user called System, was created (automatically). Please, do you know a way to not display it ?

peregrine

@pamela

in class.memberslistenhmodel.php

change line 14 from

 ->Where('Deleted',false)
to

   ->Where (array('Deleted'=>'false', 'User.Name <>' => 'System' )) 

Pamela

Thanks @peregrine but I get the following Parse error: syntax error, unexpected T_STRING in /../plugins/MembersListEnh/class.memberslistenhmodel.php on line 14

peregrine

@pamela

follow the instructions below or follow this link
http://vanillaforums.org/discussion/comment/166897/#Comment_166897

let me just put it this way:

I deleted

->Where('Deleted',false)

http://vanillaforums.org/discussion/comment/166897/#Comment_166897

->Where (array('Deleted'=>'false', 'User.Name <>' => 'System' ))

and it works perfectly for me.

Your message says syntax error - so you probably didn't do exactly as I did.
(I suspect you did something different otherwise you would not have an error)

Syntax errors usually result from missing commas, quotes, parentheses to name a few.

But this is purely a delete one line and cut and paste a line in the same exact spot.

Best I can do for you.

if you still have the problem - cut and paste your code and post it, and give me a list of every file in the directory (including ones you may have created).

Pamela

aaarrch, you 're right one more time missing to copy&paste this -> at the beginning, then I'm really sorry and thank you for your support

peregrine

no problem - at least you provide feedback at least and often you are the one who is correct.

tonex

It seems that IPEmailView does not work for any other role except Administrator, even if I check the IPEmailView checkbox of a custom role I created.

I checked the ShowEmail and ShowIP checkboxes in the Member List Enhanced Option page. I checked the GenView and IPEmailView checkboxes of my custom role in the Roles and Permissions page.

peregrine

I made it that way if you want to change it let me know what you want to do and I'll tell you how to do it. The rub is I didn't want anybody except Admins to be able to see email and ip address but I can tell you haow to customize for another role probably. Do you want to show email address and ip for the other role?

tonex

Yes, please. I need it so that any custom role can view IP and e-mail when I allow them to.

peregrine

@tonex
I'm not going to do it for you. But I will tell you what changes you could make and the outcomes. It may be insightful for you though.

I will not however take responsibility for any changes you make.

alternative 1

you will notice this statement in 4 places in memtable.php in the plugin.

CheckPermission('Garden.Users.Add')

e.g.
you could change that to CheckPermission ('Garden.Moderation.Manage')

and
anybody who has the permission moderation manage would see e-mail and ip address if the boxes are checked.

alternative 2

or you could change this statement in 4 places in memtable.
CheckPermission('Garden.Users.Add')

to

CheckPermission ('Plugins.MembersListEnh.IPEmailView')

in this case theoretically ANY role that has the IPEmailView Checked will now see ip and email addresses if you have checked the ip and email checkboxes (note this directly makes the previous documentation associated with the plugin invalid).

If I were doing it I would test first with only checking ip addresses and looking at all roles with test logins and if I was satisfied - I would add the checkbox for e-mail

so Make sure what you are doing and verify with ALL roles to make sure it works as you want it to.

currently it is set to anybody who can add users (which made sense because if you are adding users you will see ip and e-mail)

tonex

Used alternative 2 and it works. Thanks, peregrine. U Da Man!

peregrine

@tonex. Glad it worked for you. Yea it was awesome! or was it insightful.

peregrine

If you find that your rss feed breaks when using this plugin.

please remove last line from default,php that shows something like this.

?>

this will fix things.

on the next revision I will change it.

andrewborstein

Is it possible to add a default profile photo? Otherwise the table doesn't look very pretty when some users have photos and some don't—it's unevenly spaced. I'm not a php guy but I can occasionally tweak with some success. I tried this (for some reason I can't ge the code snippet box to work right):

if ($ShowPhoto)
echo '

' .UserPhoto($User,array('LinkClass'=>'ProfilePhotoCategory','ImageClass'=>'ProfilePhotoMedium')) . '';

     else
     echo '<td><img src="http://mysite.com/defaultimageurl.gif" alt="" class="ProfilePhotoMedium"></td>';

to no avail. Ideally the default image would also link to their profile.

Thanks in advance!

peregrine

@andrewborstein

Personally, I like the way it is, but could do a custom change.

if you want to sponsor and pay for custom changes - I can do it for you.