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.066 seconds
  I still don't believe he can really be human to do all this ! From all of the forums that I've ever participated in this is certainly the one that most encapsulates the feeling of being truly open source where everyone's opinions and contributions can and will shape the development of the service! It's truly awesome! Hope you enjoy cooking and look forward to reading and contributing to any of the editorial work that you proposed too!! Thanks
Gez (audibleid - JED)

Get Started