HackerOne users: Testing against this community violates our program's Terms of Service and will result in your bounty being denied.

Feedback for MembersList Enhanced

peregrineperegrine MVP
edited August 2012 in Feedback

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.

I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

«13

Comments

  • 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

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

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

  • PamelaPamela ✭✭
    edited August 2012

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

  • peregrineperegrine MVP
    edited August 2012

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

    perhaps...

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

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

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

  • peregrineperegrine MVP
    edited August 2012
    @pamela
    
    in class.memberslistenhmodel.php
    
    change line 14 from
    
     ->Where('Deleted',false)
    to
    
       ->Where (array('Deleted'=>'false', 'User.Name <>' => 'System' )) 
    

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

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

  • peregrineperegrine MVP
    edited August 2012

    @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).

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

  • 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

  • peregrineperegrine MVP
    edited August 2012

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

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

  • tonextonex New
    edited August 2012

    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.

  • 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?

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

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

  • peregrineperegrine MVP
    edited August 2012

    @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)

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

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

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

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

  • 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.

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

  • edited November 2012

    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!

  • peregrineperegrine MVP
    edited November 2012

    @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.

    I may not provide the completed solution you might desire, but I do try to provide honest suggestions to help you solve your issue.

Sign In or Register to comment.