diogo Posted May 2, 2013 Posted May 2, 2013 Ryan, can I propose that the ProcessHome class has this public function ___execute() { $this->session->redirect($this->page->child->url); } instead of this? public function ___execute() { $this->session->redirect("page/"); }
ryan Posted May 3, 2013 Posted May 3, 2013 Not sure I understand -- Why? It's a Friday here, end of the day, and my mind is lazy.
diogo Posted May 3, 2013 Author Posted May 3, 2013 to make the first page in the admin the one that appears by default. I was working on custom admin pages module, and occurred to me that PW could be flexible on this also. Although personally I really like the tree as the default view, I thinks it should be the responsibility of each one to decide about something like this. 3
diogo Posted May 4, 2013 Author Posted May 4, 2013 Ok, there is no need to change the core for this. I created a small module that allows it for those that really want to do it. https://github.com/ocorreiododiogo/ProcessHomeAdmin 5
teppo Posted May 5, 2013 Posted May 5, 2013 Actually, @diogo, your proposal made a lot of sense, just saying. Doesn't sound like a very common need, but I don't see any harm in it either.. and it would definitely open a window for clean (and "codeless") customization of admin. Thumbs up for this. 1
ryan Posted May 7, 2013 Posted May 7, 2013 This makes sense to me, I've gone ahead and changed it. It should appear on the dev branch next time I'm pushing updates to it, probably later this week. 6
Pete Posted May 9, 2013 Posted May 9, 2013 I've often thought about this and have a few projects that might need a custom landing page - dashboard anyone? 1
kongondo Posted May 9, 2013 Posted May 9, 2013 Oh no, what I have I done?! Exactly! You've opened a can of worms! Just kidding. I think there is a place for such functionality in PW alongside what other modules do. I have so many ideas with what to do with your module it is making my head spin! 1
Pete Posted September 4, 2013 Posted September 4, 2013 Finally using this in a booking system - thanks Diogo Might still be nice to make it core though ryan? I don't think it would hurt and I can't see it causing any real confusion as the only time you would want to tweak the order of the admin pages you would probably be putting in custom admin pages and want tomaybe do something like this anyway. I guess what I'm saying is it won't affect 99% of users, but would be helpful to the 1% without adding any bloat.
Pete Posted September 4, 2013 Posted September 4, 2013 Hmm... actually it doesn't seem to work. Using latest dev branch and it still defaults to the page list when another page is set as the first child of the Admin page
ryan Posted September 7, 2013 Posted September 7, 2013 I did change this before, but had to change it back as I later found it broke something else. It's been awhile now and I don't even remember what it was that was broken. One way you can get around it is to change the 'process' field on the admin homepage to your own. ProcessHome is a very simple module, so you can just copy it to your /site/modules/ProcessHomeCustom.module and change it to do what you need, then edit the admin page and make it use your version.
Pete Posted September 7, 2013 Posted September 7, 2013 Thanks ryan - I literally just copied and pasted it into site/modules 10 minutes ago with the relevant changes and it did the trick. EDIT: Though I see where I went wrong with diogo's module was not reading the instructions to change the process for the admin page
diogo Posted September 7, 2013 Author Posted September 7, 2013 Oh, I saw this only now... So, the module is working as it should Pete?
Pete Posted March 9, 2014 Posted March 9, 2014 Sorry diogo - bit of a late reply It does work if you actually follow the instructions and change the default admin page to use the Process (I was being an idiot and changing my new admin page to use the process). 1
owzim Posted May 27, 2014 Posted May 27, 2014 Great little nifty module diogo, I see some use cases for this.
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now