Handy "edit this page" link when your logged in!

[englishdingbat]

I wanted a way to quickly edit the page when I was on it, so i just added this simple bit of code in to my template. Place where ever it is convenient for you, but it only shows when you are logged in. You must have login enabled in prefs, but someone might be able to hack the code further???

Code:

<?php
 if(FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID'))) {
?><a href="<?php echo ADMIN_URL; ?>/index.php"><?php echo $TEXT['ADMINISTRATION']; ?></a><br />
<a href="http://[you-website.domain]/admin/pages/modify.php?page_id=<?php echo $page_id; ?>">Edit Page</a>
<?php
}
?>

simple i know, but useful.
Pete

[pcwacht]

Very good idea, 1 improvement though, the second url, doens't make use of the ADMIN_URL yet, change to:

Code:

<?php
       if(FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID'))) {
      ?><a href="<?php echo ADMIN_URL; ?>/index.php"><?php echo $TEXT['ADMINISTRATION']; ?></a><br />
<a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=<?php echo $page_id; ?>">Edit Page</a>
      <?php
      }
      ?>

I suggestion, maybe a  check to see if the logged in user is admin or has the rights to edit the page??

John

[baZzz]

Very handy indeed, good job!
A little improvement, in case you have a multilingual website, you might want to do this:

Code:

<?php
if(FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID')))
{
  ?>
  <a href="<?php echo ADMIN_URL; ?>/index.php"><?php echo $TEXT['ADMINISTRATION']; ?></a><br />
  <a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=<?php echo $page_id; ?>"><?php echo $HEADING['MODIFY_PAGE']; ?></a>
  <?php
}
?>

[felix_se_cat]

check to see if logged in user is an admin could be done with:
if ($wb->get_session('GROUP_ID')==1)

(i did so and it works fine for me)

[Vincent]

Very handy indeed!

I slightly refined it (open new window). Thanks.
Vincent

Code:

<?php
if(FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID')))
if ($wb->get_session('GROUP_ID')==1)
{
  ?>
  <a href="<?php echo ADMIN_URL; ?>/index.php"target="_blank"><?php echo $TEXT['ADMINISTRATION']; ?></a><br />
  <a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=<?php echo $page_id; ?>"target="_blank"><?php echo $HEADING['MODIFY_PAGE']; ?></a>

[marathoner]

This works great for me and saves time when tweaking pages.

However, I'm having a problem. This snippet works fine for every page except my 'home' page which happens to be $page_id=1. Has anyone else had a problem using this snippet with $page_id=1? If so, is there a solution?

[dellington]

I love this! This will be really great for some of the folks I build websites for. Thanks!!

[DGEC]

I was trying to figure out why my implementation of this didn't work until I realized that I was on Home as well. I guess we'll just have to hardcode IF id="" then ID=1 or whatever your home page code is (mine is 3, must have deleted it twice I guess)

Vincent - good idea, but your code got cut off at the end. 

So here's my combined solution!

Code:

<?php  
if (FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID')))
 if ($wb->get_session('GROUP_ID')==1)
 {
  ?>
  <a href="<?php echo ADMIN_URL; ?>/index.php"><?php 
      echo $TEXT['ADMINISTRATION']; ?></a><br />
  <a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=<?php 
      if ($page_id) 
         echo $page_id; 
      else 
         echo "3";
      ?>" target="_blank"><?php echo $HEADING['MODIFY_PAGE']; ?></a>
  <?php
     }    
    ?>     

[icouto]

Wow! This hint is SOOO useful!

Is there a way to automatically find out what the id of the home page is without having to hard-code it? I ask because I can imagine some of my clients deleting the home page, making another one, and causing the link to fail.

[DGEC]

You mean... like I did? 

That's a good point. You might have to just tell them not to do that, I couldn't figure out another way to do it without mucking into the core further and I didn't have time to do that.

It would be best to integrate this function into the system, and have it as an option on Settings, like Log In and Search.

Actually another thing I've been wanting to try doing is a "view page as guest/member/non-admin" function. I remember seeing that on another CMS and thinking how useful that would be. They implemented it as a fixed administrator menu option.  That method sounds like a lot of work for WB.

One quick mod might be to change the Edit Page link to a drop-down listbox with Edit as the first option, and "View as Guest,..." and then display the rest of the types in the Group database.  The view link could pass a parameter and a refresh would then automatically bring back the edit/view menu, hopefully.  Similar to the template preview function - it can work for one page view, but harder to make it sticky.  If it was an available value like the test if you're signed in or not, then you could turn it on and off.

[gavjof]

Thank for this 'handy' piece of code.
I'd used something similar in the past for jumping to the ADMIN_URL but the "edit this page" feature is really nice

[vivitron]

Greetings,

I was using this and made a small modification that makes the link always for the default page when the id changes.

Code:

<?php  
if (FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID')))
 if ($wb->get_session('GROUP_ID')==1)
 {
  ?>
  <a href="<?php echo ADMIN_URL; ?>/index.php"><?php 
      echo $TEXT['ADMINISTRATION']; ?></a><br />
  <a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=<?php 
      if ($page_id) 
         echo $page_id; 
      else 
         echo  $wb->default_page_id; ;
      ?>" target="_blank"><?php echo $HEADING['MODIFY_PAGE']; ?></a>
  <?php
     }    
    ?>     

Notice the use of  $wb->default_page_id. 

Hope this helps!

Sincerely,
Dustin Lambert

[DGEC]

So that's how it's done.

Thanks for that info - good job, Dustin!

[icouto]

Dustin, that is great!

I might include that in the upcoming Template Primer - a guide for WB Template Designers.

[Mesum]

When I add this code to theme, it won't show... Where else can I add it?

[icouto]

@Mesum: hmmm, hard to know what could be the cause. Here are a few suggestions for you to check and try out:

1) make sure that you are adding the code to the template's 'index.php' file of the template you are currently using in your site.

2) make sure that you are adding the code to a section/element in the BODY of the template, and that it is visible - and not somewhere that is a hidden part of the template's layout.

3) make sure you have 'Login enabled' in your Admin->Settings, and that you login!

[Mesum]

AHA! I didn't have 3 on Thanks icouto.

[Panther]

Right now, this only works if someone is an admin...

how could this be tweeked to check the admin_groups value in the pages table to see if a user has been enabled as an editor?

I have people who are editors of specific pages. I could hard code this something like...

if ($wb->get_session('GROUP_ID')==1) | ($wb->get_session('GROUP_ID')==3) {

But that only works if the group of editors is editors for every page...

ideally, doing a check of the DB would be slick so it checks and only shows up an edit option if you are authorized to edit that particular page.

Side note to this, I modified it a bit to show the "edit page" icon from the admin panel in the upper right corner of my content area when logged in..

Code:

<div id="content">
<?php  
if (FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID')))
if ($wb->get_session('GROUP_ID')==1) {
?>
<a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=<?php 
if ($page_id) 
echo $page_id; 
else 
echo  $wb->default_page_id; ;
?>" target="_blank"><img align="right" src="<?php echo ADMIN_URL ?>/images/modify_16.png" alt="<?php echo $HEADING['MODIFY_PAGE']; ?>" /></a>
<?php } ?>     
<h2><?php echo MENU_TITLE; ?></h2>
<?php page_content(); ?>

I placed it just inside my content div container, before any other headers or content.

[Panther]

ok, this works... mostly (I looked in the admin/pages/settings.php file to find where it checks permissions)

Code:

<?php  
if (FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID')))
// Get permissons
$database = new database();
$results = $database->query("SELECT * FROM ".TABLE_PREFIX."pages WHERE page_id = '$page_id'");
$results_array = $results->fetchRow();
$old_admin_groups = explode(',', $results_array['admin_groups']);
$old_admin_users = explode(',', $results_array['admin_users']);
$this_user = $wb->get_session('GROUP_ID');
if (is_numeric(array_search($this_user, $old_admin_groups)) ) {
?>
<a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=<?php 
if ($page_id) 
echo $page_id; 
else 
echo  $wb->default_page_id; ;
?>" target="_blank"><img align="right" src="<?php echo ADMIN_URL ?>/images/modify_16.png" alt="<?php echo $HEADING['MODIFY_PAGE']; ?>" /></a>
<?php } ?>     

But, the icon shows up on any page that doesn't have a specific $page_id, like the default page... even when I'm logged out.. and I'm not sure why.

[pcwacht]

Check the use of { and } for the nested if's
I don't see a { at the first if  and think you need an extra ending } at the last php bit.

Good luck

John

[Panther]

Check the use of { and } for the nested if's
I don't see a { at the first if  and think you need an extra ending } at the last php bit.

Good luck

John

That did it.. there were actually a couple spots in the original snippet that did not have {}s

here's what I came up with and it seems to be working fine. This does NOT put the link to the admin home page like the original snippet, it puts an image link, right now on the right side (but that can be changed). I placed mine at the top of my content block, just before the page title so it puts the icon across from the title.

You can just as easily include this at the bottom of your content or elsewhere on your page.

I tried to package this as a snippet-module like the RandomImage one but doing so broke things, it did not like the session calls. I may try to fix that eventually... but for now, this checks the permissions for each page, and if someone is logged in, and has permission to edit that particular page, the link shows up. If they don't.. it doesn't show up.

Code:

<?php  
if (FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID'))) {
// Get permissons
if ($page_id) { $this_page = $page_id; }
else { $this_page = $wb->default_page_id; }
$database = new database();
$results = $database->query("SELECT * FROM ".TABLE_PREFIX."pages WHERE page_id = '$this_page'");
$results_array = $results->fetchRow();
$old_admin_groups = explode(',', $results_array['admin_groups']);
$old_admin_users = explode(',', $results_array['admin_users']);
$this_user = $wb->get_session('GROUP_ID');
if (is_numeric(array_search($this_user, $old_admin_groups)) )
{
?>
<a href="<?php echo ADMIN_URL; ?>/pages/modify.php?page_id=
<?php 
if ($page_id) { echo $page_id; }
else { echo  $wb->default_page_id; }
?>
" target="_blank"><img align="right" src="<?php echo ADMIN_URL ?>/images/modify_16.png" alt="<?php echo $HEADING['MODIFY_PAGE']; ?>" /></a>
<?php } ?>     
<?php } ?>

[DGEC]

Ha! Looks like we both had the same ideas. 

I ran into issue making a snippet too and stopped work on it.

Guess this will work go enough though - THANKS!

[kickarse]

Remember in order for this to work you must have Login enabled.

[pcwacht]

If you don't have login enabled,
change this:

Code:

if (FRONTEND_LOGIN == 'enabled' AND is_numeric($wb->get_session('USER_ID'))) {

to:

Code:

if (is_numeric($wb->get_session('USER_ID'))) {

[DGEC]

That's a good idea, that way you don't have to have a login page tempting people on every page.

Hmm....

I tried to make a "member tools" page that includes a link to the login page (used the id=#) but it gets a 404?  Odd. What's going on here...