ranchhand6

That was it. I apparently had a card.php that was extremely old. I copied the one from wp-content/plugins/connections/templates/card/ to wp-content/connections-templates/card/ and that seemed to solve the problem.

I didn’t diff the rest of the directory contents. Would it be advisable to take the rest of the files in wp-content/plugins/connections/templates/card/ and replace the old ones as well?

My pleasure!

I synchronized my fork to your /develop branch, so I see where you are. With regards to the API, what can I do to help that keeps me out of your way?

I finally resolved it down to a transaction isolation problem between tests in the same file. I was adding a mocked entry in one test and was able to retrieve it in another so I assumed that I would be able to update it as well. This is not the case. The test framework apparently prevents this. When I added the mocked entry inside the test case I was trying to do the update in, everything started working as expected.

Write it up to my inexperience with WP_Unit.

I’m using the id from the add immediately prior, looking at the $response object and getting the id from there and putting it into the update.

However, not seeing the entry in the database after the test. I suspect this is because the test framework is using transactions to isolate the test data, but am not deeply familiar enough with WP_Unit as yet to know if this is true. Is there a caching layer I need to flush after the add?

Not sure if this is the right place to ask a programming question, if there’s a different way to communicate, I’m more than happy to use whatever means you are comfortable with. Also, I haven’t been doing PHP for awhile now, so apologies is this error resulted from simple oversight.

We decided to go ahead and put a little work into the REST API interface, so I’ve forked your github repo and have been working my way forward using WP_Unit tests. Please consider everything done to date as draft, we are not married to any of it.

I have a question about an interaction I’m seeing between class.entry-data and class.actions. We’re mocking some data into an entry, and the thing we’re trying to do is to attach an image in the WP_Unit test so that we can test and debug the output of the REST API. The problems seems to be that for everything _but_ the image, the mocked data seems to work fine but with the image data the “update” action fails. While exploring this, we noticed the following:

At class.entry-data->update():5003 (or so), after submitting the CN_ENTRY_TABLE update, you use the following check before allowing the remainder of the tables to update:

       if ( FALSE !== $result ) {

In our test case, this succeeds because the rest of the updates work. The variable $result is returned later in the method as the result of the entire method.

However, at class.entry-actions->process():789 (or so), you use the following check on the return:

	if ( $entry->update() == FALSE ) {

In our case, this fails.

Indeed, when we put some testing code into the method,

    
    echo "After cn_update-entry ". ((FALSE !== $result) ? "true" : "false" ). " \r\n";
    echo "After cn_update-entry ". ((FALSE == $result) ? "FALSE" : "TRUE" ). " \r\n";

… we get these results:

After cn_update-entry true
After cn_update-entry FALSE

This seems a little counter-intuitive.

I hesitate to go much deeper into the base code without consulting with you. I have a branch and a test that stimulates the issue, if you care to look at it. Again, what we’re trying to do is to attach an image to a test entry in order to test the output, if you have an easy way to do that without looping through the class.entry-actions->add() and class.entry-actions->update() we’re more than happy to take a different approach.

Thanks.

AWD

• This reply was modified 9 years, 6 months ago by ranchhand6.

Ok, we’ll give that a try.

Thanks for the help!

Tracked down through the php.ini. These values look right to me:

error_log = /var/log/php5-fpm.log
error_reporting = E_ALL & ~E_DEPRECATED & ~E_STRICT
display_errors = Off
display_startup_errors = Off
log_errors = On
log_errors_max_len = 1024
ignore_repeated_errors = Off
track_errors = Off
xmlrpc_errors = 0
xmlrpc_error_number = 0
html_errors = On

etc.

Is there a setting inside WP that might be setting the error level lower at runtime?

Thanks for responding.

This is was a little interesting to run to ground. I don’t have shell access onto the server, only cPannel.

First, MySQL. Through PHPMyAdmin:

SELECT @@global.time_zone , @@session.time_zone;

@@global.time_zone  @@session.time_zone
SYSTEM              SYSTEM

So it appears as if MySQL is using the same timezone that’s set on the server.

From phpinfo();

Warning: phpinfo() [function.phpinfo]: It is not safe to rely on the system’s timezone settings. You are *required* to use the date.timezone setting or the date_default_timezone_set() function. In case you used any of those methods and you are still getting this warning, you most likely misspelled the timezone identifier. We selected ‘America/Los_Angeles’ for ‘PDT/-7.0/DST’ instead

“Date” section of phpinfo();

date/time support	                enabled
"Olson" Timezone Database Version	2013.3
Timezone Database	                internal
Default timezone	                America/Los_Angeles

This is a little strange, but probably Ok. I didn’t set a timezone when I ran phpinfo(), so php selected one for me based on where it thought it was. Ok, it was probably right.

Again, from phpinfo();

“PHP Variables”

_SERVER["REQUEST_TIME"]	            1395423577

Aha!

UNIX time 1395423577 is 03/21/2014 5:39pm GMT.

This translates into 10:39 am PDT, which is when I made the request.

So it seems like the unix timestamps on the server are correct.

To confirm, if I do a

SELECT UNIX_TIMESTAMP( TIMESTAMP )

on the field in question, it puts out a GMT time that matches the value that I see in the report.

I agree that the date_i18n( $format, $time ) should do it, but it doesn’t appear to for some reason.

Any ideas or suggestions appreciated.

T