tvEditor beta 0.8.5

Hi Heliotrope!
Thanks for this great module! I just installed it (0.8.5.2) and it works perfectly, no functional problems encountered up to now. It just makes the handling of lists much easier - you could even allow clients to edit the values in a comfortable way.
... and all the search-functions will really make life easier ...

Just some small suggestions:
- usually with listbox-options just the value-field is shown. This changes as soon as one value look ’s like "name == value". This is great ... But: it would be even greater if there would be an button to add the "value" column, or, if it allways would be shown as "value, if different from name" and
- if only the name is filled in without a value, the name should be used
- currently only "listbox,listbox-multiple,checkbox,option,dropdown"-TV´s are supported. it would be great, if the other TV´s would be shown as well. of course, most functions of the module are list-functions, but still it would be perfect to be able to have the others as overview as well ...

BUT: great module! It really makes MODx easier and faster to use ...

Hi,

thx for your feedback

- usually with listbox-options just the value-field is shown. This changes as soon as one value look ’s like "name == value". This is great ... But: it would be even greater if there would be an button to add the "value" column, or, if it allways would be shown as "value, if different from name" and
- if only the name is filled in without a value, the name should be used

Not sure to understand correctly :-(
MODx offers two ways to write tvs options.
tveditor does not offer the possibility to mix the syntax (value||value2 or name==value||name==value)
you have to choose one of them when you add the first options.

- currently only "listbox,listbox-multiple,checkbox,option,dropdown"-TV´s are supported. it would be great, if the other TV´s would be shown as well. of course, most functions of the module are list-functions, but still it would be perfect to be able to have the others as overview as well ...

That’s a bit hard to do !
You can use tvexplorer for this
It allows to render any kind of tv (except file).

I could add a feature to allow you to code your form (as in tvexplorer) and call them in the backend.

:-)

Searching /listing documents by TV value etc, installed flawlessly on 0.9.6; I would say this instantly bumps it to 0.9.6.7 !!

I believe there is an issue regarding TVs "name == value" in the current MODx core, it may affect further development of tveditor, please check FlySpray#911

- usually with listbox-options just the value-field is shown. This changes as soon as one value look ’s like "name == value". This is great ... But: it would be even greater if there would be an button to add the "value" column, or, if it allways would be shown as "value, if different from name" and
- if only the name is filled in without a value, the name should be used

Not sure to understand correctly :-(
MODx offers two ways to write tvs options.
tveditor does not offer the possibility to mix the syntax (value||value2 or name==value||name==value)
you have to choose one of them when you add the first options.

it is not really a problem ... but when testing (see the screenshot in my last post) I didn´t fill all values - therefore I had two names associated with a value and the others were just blank values - it would be great, if the default value (if nothing is filled in) would be the name ... it would be great, if the output would look like "name==value||name==name||name==name" or "name==value||name||name" instead of "name==value||name==||name==" if the 2nd and 3rd value are blank ...

I could add a feature to allow you to code your form (as in tvexplorer) and call them in the backend.

this would be great!!!!!!!! I would love to add a preview of some text-tv´s to the document-overview ... and thumbs of the selected images ;-)

Just now a new warning occured in tvexplorer:
Warning : Wrong parameter count for explode() in /www/htdocs/v158413/modx096/assets/snippets/ditto/extenders/tve.extender.inc.php on line 49
... I was deleting some TV´s, the warning was shown when I had just one list-TV without values, and now I can get rid of it ... before it worked fine.

And a question: in tvExplorer I already set the StartID. How can I change it now?

I just love your work! If somebody implements the functions of the current "Resources - Template Variables" - section it could just replace it - and bump MODx it beyond 0.9.6.7 ;-)

it is not really a problem ... but when testing (see the screenshot in my last post) I didn´t fill all values - therefore I had two names associated with a value and the others were just blank values - it would be great, if the default value (if nothing is filled in) would be the name ... it would be great, if the output would look like "name==value||name==name||name==name" or "name==value||name||name" instead of "name==value||name==||name==" if the 2nd and 3rd value are blank ...

That’s weird
You should not be able to save the options when the form contains empty fields.
I will check this out.

And a question: in tvExplorer I already set the StartID. How can I change it now?

click on any label in document tree, the value will change accordingly

:-)

P.S about new features, I have been working for about 9 months on the module and the snippet i do not promise anything. I don’t have too much free time to keep working on this
I think I will reauthor most of the code for 0.9.7

You should not be able to save the options when the form contains empty fields.

Just in case you want to reproduce it:
I had a TV with values only (NOT name - value) and inserted "x==y" as value - and it switched to the name & value view ... with empty values (except one). ok, this was not the common way of using it ...

click on any label of the folder in document tree, the value will change accordingly

great!

... and I just wanted to say, that I love the direction of this module - and I see a promising picture of where MODx could be somewere soon ...
(now I should start to contribute to the community ... but I am still learning, so it takes a little bit ...)

Just in case you want to reproduce it:
I had a TV with values only (NOT name - value) and inserted "x==y" as value - and it switched to the name & value view ... with empty values (except one). ok, this was not the common way of using it ...

You ’re pushing the module limit

When the elements are parsed (name==value||... or value||...)
tvavatar class apply a mode to the tv.
Mode 1 is value||.. and mode 2 is name==value||...
the mode selection is based on the condition that elements contains ==

I haven’t thought about this case (enter name==value in value field).

I could add a control on submitted values.

Thx again for your feedback and testing.

:-)

what about just adding the line "if ($value == "") $value= $name;"?
it would allow leaving the value-field empty and using the name-field-value instead ...

I’m gonna try to fix this.
I have to check first if there could be side effects.

I guess I will prohibit the "==" in options value and will add a switch type feature (mode 1<=>mode 2) .

But you ’ll have to be patient

By the way, did you try the tvexplorer snippet ?

not yet - I don´t use tagged pages up to now, so the snippet doesn´t make sense with these. but if I have some time I will try it!