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.112 seconds
  Love COOKING with COOK It has all the ingredients for a delicious Joomla component!
audibleid (Forum)

Get Started