Textpattern CMS support forum
You are not logged in. Register | Login | Help
- Topics: Active | Unanswered
Re: Make recent_articles context aware (exclude self)
ruud wrote:
Some discussion between the developers, tonight :)
Any update?
Offline
Re: Make recent_articles context aware (exclude self)
ruud wrote:
Some discussion between the developers, tonight :)
Still waiting for an update :)
Offline
Re: Make recent_articles context aware (exclude self)
Sorry for the delay.
So far it looks like this won’t be accepted into the stable 4.0.x branch (no new features or attributes, just bugfixes), so it’ll have to wait for the 4.1 release. It’s not the final answer, but that’s all I can tell you right now.
Offline
Re: Make recent_articles context aware (exclude self)
ruud wrote:
Sorry for the delay.
So far it looks like this won’t be accepted into the stable 4.0.x branch (no new features or attributes, just bugfixes), so it’ll have to wait for the 4.1 release. It’s not the final answer, but that’s all I can tell you right now.
Ok. Do you have a version 4.1 CVS tree that you can check it into, or?
Offline
Re: Make recent_articles context aware (exclude self)
Yes, we do… but I wonder if we should implement it like that or if your suggested behaviour should be the default behaviour, because ‘recent_articles’ as it is now can probably be emulated by using article_custom.
Last edited by ruud (2007-08-21 14:55:20)
Offline
Re: Make recent_articles context aware (exclude self)
Good work gemal.
For heavens sake, devs, put this in crockery already. You’ve got a person creating a solution to a long standing problem, and you guys have to sit on it like it’s going to hatch or something… and you’re going to piss-off and discourage motivated contributors.
I don’t understand the huge deal. If it’s a problem revert it. If not, good! I read the change log on crockery, it’s not as if there haven’t been a few “er, sorry ‘bout that revisions”.
Offline
Re: Make recent_articles context aware (exclude self)
Crockery is a chance to do it right, to fix default behaviour that cannot be changed in the stable 4.0.x branch. I personally think that what Henrik suggests (not showing the current article) should be done by default, not even requiring a new attribute. That would make it a one line patch in crockery, but since nobody else is commenting on what the default behaviour should be, I’m not yet doing so, because my opinion on this may not necessarily be that of the TXP team or the users.
This is not to annoy or discourage contributors.
Offline
Re: Make recent_articles context aware (exclude self)
ruud wrote:
since nobody else is commenting on what the default behaviour should be
just wanted to say that i share your view on recent_articles / article_custom and think that yes, it should be the default behavior.
Offline
Re: Make recent_articles context aware (exclude self)
OK, here’s my opinion… don’t show it by default. Add an attribute to toggle showself, because there will be instances where people will want to show that article.
I know your intent is not to annoy or discourage, I think the devs do excellent work. I’m just saying that when some motivated guy submits a patch and after promising that you’ll look at it that night, it deserves more than a tepid response two weeks later.
Offline
Re: Make recent_articles context aware (exclude self)
i am no programmer but whats the reason for it not to make it into 4.0.x? as far as i can tell it shouldn’t break current functionality because it would need the optional attribute to kick this patch into action.
i only ask because i have no idea as to the development stage of crockery. i took a peak at the dev trac and even tried out the svn version which came up with errors galore (understandable as its experimental). as a non-coder i basically have to rely on plugins/patches like these as i’m unable to alter textpatterns core to their needs like the seasoned pros here can.
Offline
Re: Make recent_articles context aware (exclude self)
Although 4.0.x has seen some new tags and attributes in the past versions, the idea is to do bugfixes only in 4.0.x (a new attribute isn’t a bugfix) and focus all new development on crockery/4.1. Having said that, don’t be surprised if it shows up in 4.0.x anyway…
Offline
#24 2007-08-21 20:07:08
- diuk
- Member
- Registered: 2007-08-21
- Posts: 15
Re: Make recent_articles context aware (exclude self)
This sounds like a great idea, thanks a lot gemal for taking the time to implement it. I think this type of contribution should be encouraged. Keep up the good work, guys!
Offline