TVs + NewsPublisher?

[Guest]

Welcome, Guest. Please login or register.
Did you miss your activation email?

1 Hour 1 Day 1 Week 1 Month Forever
Login with username, password and session length

[Mark]

Is there any way to set a tv using the new NewsPublisher?

[xwisdom]

Is there any way to set a tv using the new NewsPublisher?

You mean the set the value of a TV?

[Mark]

Yeah.

[xwisdom]

Yeah.

It currently does not but you can modify it to do so.

It's best to make a copy of the snippet and give it a new name to prevent upgrade problems

[The Man Can!]

In attempting to figure this out, I've hit a bit of a stumbling block. There doesn't seem to be an API function to write to a TV?  I see that there is a $modx->putChunk, and I thought there'd be something similar for TVs, but I don't see anything in the documentation or in the document.parser file. I'm guessing I'm wrong, as I almost always am.

Thinking that perhaps TVs can just be added with the regular document info (like longtitle, alias, etc.), I tried adding the TV values along with the other values in the $flds array (around line 190), but can't get anything going there either. 

Any advice on which direction to go to get this working?  Thanks for any help!
Brett

[rthrash]

Hi Brett,

Unfortunately at this time there is no API for writing to TVs as you've discovered. You'll need to create a query via the DB API in order to do it.

However, this would be a great time to log a request to the bugtracker for just such a function as we're in that "mode" right now.

Thanks!

[The Man Can!]

Thanks Ryan.  I logged that feature request in Flyspray (after spending 5 minutes trying to find the "Create new user" link up at the top ).  I'll attempt a query via the DB API like you suggested and will update if I have success.

[jaredc]

Man I'm glad I'm not the only one. After the instructions said look at the BOTTOM for the new user account link, I looked and looked. Thought my lasik surgery was letting me down. Then I finally did a search and found it smack at the TOP. 

[OpenGeek]

My apologies for your time wasted searching the bottom of the page!    Flyspray moved the login and registration to the top of the page since the last version, and I neglected to change the wording on the page to reflect this.

[Djamoer]

Happened to me too...
Hehehee...

But that's the fun of open source community, we learn something from error/mistake of other people too... I'm just trying to point out that it's something that makes open source project fun, compare to corporate business project like what I'm working on right now. A small mistake in typo is being taken VERY serious.

So anyway, looking forward to get to know y'all...

C ya...

[jlarson]

In the meantime I've hacked the following function into document.parser.class.inc.php:

Code:

        #  Set template variable value
        function setTV($tvname, $value, $docid="") {
       if($tvname== "") return false;
               if($docid == "") $docid = $this->documentIdentifier;
               //get id of tv
               $result=$this->db->select("id", $this->getFullTableName("site_tmplvars"), "name = '$tvname'");
               if (!$result) return false;
               $row=$this->fetchRow($result);
               $rows_affected = $this->db->update("value='$value'", $this->getFullTableName("site_tmplvar_contentvalues") , "tmplvarid=".$row['id']." AND contentid=".$docid);
               return $rows_affected;
        }
       

Of course if you just replace each "$this" with '$modx' you could put it in a snippet.

[The Man Can!]

Oh my gosh I could kiss you for that!   
Seriously, thank you so much.

[davidm]

+1

Thanks !!!

Oh my gosh I could kiss you for that!   
Seriously, thank you so much.

[The Man Can!]

Hi all.
Sorry for the super long delay in this post, but that post above doesn't quite work (at least, not from what we could figure).  Luke used that as a starting point and made this, which seems to work absolutely beautifully:

There's a feature/bug tracking request here:
http://modxcms.com/bugs/task/129

Code:

// HACK **********************************************************
// Insert this about line 1700 in document.parser.class.inc.php
// right after function getTemplateVarOutput
#  Set template variable value
function setTV($tvname, $value, $docid="") {
   if($tvname== "") return false;
   if($docid == "") $docid = $this->documentIdentifier;
   //get id of tv
   $result=$this->db->select("id", $this->getFullTableName("site_tmplvars"), "name = '". $tvname . "'");
   if (!$result) return false;
   $row=$this->fetchRow($result);
   // query the db to see if there are any records to update
   $result=$this->db->select("Count(*) AS RecordsExist", $this->getFullTableName("site_tmplvar_contentvalues") , "tmplvarid=".$row['id']." AND contentid=".$docid);
   $row2=$this->fetchRow($result);
   if (!$row2['RecordsExist']) {
$fields = array(
"tmplvarid" => $row['id'],
"contentid" => $docid,
"value" => $value
);
   $rows_affected = $this->db->insert($fields, $this->getFullTableName("site_tmplvar_contentvalues"));
   } else {
   $rows_affected = $this->db->update("value='" . $value . "'", $this->getFullTableName("site_tmplvar_contentvalues") , "tmplvarid=".$row['id']." AND contentid=".$docid);
   }
   return $rows_affected;
}
// END HACK **********************************************************

We made a modified NewsPublisher snippet as well that uses this.  If anybody wants to see that, just let me know.  It has some stuff hardcoded, so it's not really great for a general release, but if anybody wants to take it and run with it it shouldn't be too hard.

Best,
Brett

PS:  jlarson, if your's works, sorry! We (Luke and I) didn't have any luck with it, but maybe it's just because we goofed something.     If they both work, I guess that's great too!

[luke.stokes]

Yeah, just to clarify, I had to add a check to see if the TV had any current value yet. If not, the UPDATE query doesn't do anything so you have to do an INSERT query instead. I think MODx doesn't create blank TV entries, but only creates them when you enter a value.

Hope this helps someone out there.