Welcome, Guest

TOPIC: [ADDED] config paramer : TODO tags

[ADDED] config paramer : TODO tags 16 Sep 2012 18:16 #3689

VeCrea Offline Platinum Member Absolute JCook fan Posts: 473 Thank you received: 100 Karma: 30	Dear admin, Could you please remove this (imho useless) code... <?php /* TODO : REMOVE ME */ echo JDom::_('html.fly.cook.todo', array( 'file' => __FILE__, 'align' => 'left' )); ?>
	The administrator has disabled public write access. The following user(s) said Thank You: Tomaselli

Re: Useless code to remove, please 19 Sep 2012 10:05 #3762

admin
Offline
Administrator
Chef
Posts: 3711
Thank you received: 987
Karma: 140

It is usefull to get the template file name.

Anyway, I removed it from your generated component.
In the sandbox, it is still visible, it can be a nice help. I personally use it often.

Now it is not anymore downloadable.
If somebody ask for it, I can put an option.

The idea was simple : Remember wich layout you have treated, and wich ones are not treated.
I think every template file has to be arranged a little bit (at least in front)
So when you start working on the template, you remove the TODO marker.

What is possible to remove it everywhere, is to return null in its JDom build function.
But at the end, it result a stupid void call.

Well.
Suppressed.

Coding is now a piece of cake

The administrator has disabled public write access.

Re: Useless code to remove, please 19 Sep 2012 17:33 #3788

VeCrea Offline Platinum Member Absolute JCook fan Posts: 473 Thank you received: 100 Karma: 30	I believe it would have been good to have an option for it. I understand the goal of what you were trying to do, and actually, it was a good idea for the people that need it to help them with templating, but not for advanced users... The more I think about it, the more i think it could be great as an option. Thanks
	The administrator has disabled public write access.

Re: Useless code to remove, please 19 Sep 2012 19:38 #3792

dyvel Offline Elite Member Posts: 200 Thank you received: 11 Karma: 10	An option could be to add a "debug" parameter in component settings that will trigger this function...
	The administrator has disabled public write access.

Re: Useless code to remove, please 03 Oct 2012 13:03 #4226

admin Offline Administrator Chef Posts: 3711 Thank you received: 987 Karma: 140	Done Added in component configuration. Default : NO
	Coding is now a piece of cake The administrator has disabled public write access. The following user(s) said Thank You: JoomGuy

Forum

Ideas & Suggestions

Added functionalities

[ADDED] config paramer : TODO tags