Welcome, Guest
Username: Password: Remember me
  • Page:
  • 1

TOPIC:

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

  • VeCrea
  • VeCrea's Avatar Topic Author
  • Offline
  • Platinum Member
  • Platinum Member
  • Absolute JCook fan
  • Posts: 473
  • Thank you received: 100
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 following user(s) said Thank You: Tomaselli

Please Log in or Create an account to join the conversation.

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

  • admin
  • admin's Avatar
  • Offline
  • Administrator
  • Administrator
  • Chef
  • Posts: 3711
  • Thank you received: 986
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

Please Log in or Create an account to join the conversation.

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

  • VeCrea
  • VeCrea's Avatar Topic Author
  • Offline
  • Platinum Member
  • Platinum Member
  • Absolute JCook fan
  • Posts: 473
  • Thank you received: 100
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

Please Log in or Create an account to join the conversation.

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

An option could be to add a "debug" parameter in component settings that will trigger this function...

Please Log in or Create an account to join the conversation.

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

  • admin
  • admin's Avatar
  • Offline
  • Administrator
  • Administrator
  • Chef
  • Posts: 3711
  • Thank you received: 986
Done

Added in component configuration.
Default : NO
Coding is now a piece of cake
The following user(s) said Thank You: JoomGuy

Please Log in or Create an account to join the conversation.

  • Page:
  • 1
Time to create page: 0.153 seconds

Although I found this tool some time ago I just started learning experimenting and building this first component on Monday and now on Thursday afternoon it's already online. Although I already had a working version done with another tool this is much easier to build and at the end you have the freedom to own your own component. Once you know how to use Cook you will boost your productivity as the learning curve is really moderate compared to many other tools.
Giori (Forum)

Get Started