Tracy\Bar's output can be disabled (closes #82)

[hranicka]

Closes #82

[JanTvrdik]

+1, although public $enabled = TRUE is probably enough

Btw: the commit message should probably be: "Bar: output can be disabled [Closes #82]"

[JanTvrdik]

Both functions take no arguments

[hranicka]

Thanks @JanTvrdik , I've changed the commit message and fixed PhpDoc.

---

@JanTvrdik
I prefer public methods (::disable/::enable) instead of public property because in the future Bar can be interface and PHP interface cannot define public properties.

But I can change it to only one method eg. public function enable($enabled = TRUE).
What do you think about it?

[dg]

What about simple method hide?

[hranicka]

I've replaced the two methods with one hide($hidden = TRUE).

But "disable" sounds me better than "hide"... maybe...

And I also thinking about option for enabling the Bar again when is disabled. Maybe it's not necessary to have this option.

[dg]

Maybe disable sound better… The hide was only idea to discuss.

[EdaCZ]

Whats the status of this, please?

[bazo]

please merge this. disable/enable or hide/show are both good names. but it's a pain in the ass to develop a rest api with the bar output in the response

[dg]

@bazo Send the correct Content-Type header.

[bazo]

ok, that helped. but still, this can be a useful feature

[encero]

+1, i am already using forked version with this merged.

[hranicka]

Wait for v2.4 :)