Cumulus Support

sfws wrote: Maybe you have created a file but the script does not exist in it. Maybe the encoding of the page does not match what is declared as the encoding.

sfws wrote:If your top ten php file was loading properly, I would be able to read the source using http://kesis.info/Saa/Top10.php?view=sce, but as that is also blank, I'm pretty sure that ... your process for uploading the script has failed

mcrossley wrote:To stop hi-jacking the other thread, I thought I 'd start a new one for the Top 10 page.

You can see my version here.

The source code is here.

This was inspired by Dans (captzero - brisbaneliveweather) pioneering work, though I developed my code independently.

$mtime = microtime(false);
$mtime = explode(" ",$mtime);
$mtime = $mtime[1] + $mtime[0];
$starttime = $mtime;

$starttime = microtime(true);

// Ver 1.5 02/10/17 - Fix sql_mode=only_full_group_by errors

// Ver 1.1 23/11/15 - Removed least number of frost days and replaced with most consecutive frost days
// Ver 1.2 31/10/16 - Checks for no data in the frost records results
// Added Hemisphere variable, so frost records work south of the equator
// Ver 1.3 06/11/16 - Bit of tidying up of dataArrays - removed unused class name

// Ver 1.1 23/11/15 - Removed least number of frost days and replaced with most consecutive frost days
// Ver 1.2 31/10/16 - Checks for no data in the forst records results
// Ver 1.3 18/05/17 - Minor table header changes
// Ver 1.4 02/10/17 - Disable "sql_mode=only_full_group_by" if it is set (MySQL > 5.7.5)
// Ver 1.5 02/10/17 - Fix sql_mode=only_full_group_by errors

sfws wrote: Mark,
I've obviously looked at your code for my post this morning.
It may only be a very trivial point, but I note you use

Code: Select all

$mtime = microtime(false);
$mtime = explode(" ",$mtime);
$mtime = $mtime[1] + $mtime[0];
$starttime = $mtime;

when the same could be achieved in a single instruction:

Code: Select all

$starttime = microtime(true);

I have checked and this has been available since php4, so it is not an option that has only become possible recently.

sfws wrote: I'm also trying to work out your changes in your version 1.5

// Ver 1.5 02/10/17 - Fix sql_mode=only_full_group_by errors

because I regressed back to an earlier MySQL version because I experienced those errors in several of my scripts and, just from reading the MySQL manual, did not find a way to eliminate them whilst keeping the functionality of my scripts. So the answer would be useful to me, and I was unable to pick it up by doing a version compare, as you only have the latest version on your site, and that is where your original post above links. I did find a

// Ver 1.1 23/11/15 - Removed least number of frost days and replaced with most consecutive frost days
// Ver 1.2 31/10/16 - Checks for no data in the frost records results
// Added Hemisphere variable, so frost records work south of the equator
// Ver 1.3 06/11/16 - Bit of tidying up of dataArrays - removed unused class name

on another site, but you must have a version control problem as that does not match your

// Ver 1.1 23/11/15 - Removed least number of frost days and replaced with most consecutive frost days
// Ver 1.2 31/10/16 - Checks for no data in the forst records results
// Ver 1.3 18/05/17 - Minor table header changes
// Ver 1.4 02/10/17 - Disable "sql_mode=only_full_group_by" if it is set (MySQL > 5.7.5)
// Ver 1.5 02/10/17 - Fix sql_mode=only_full_group_by errors

so I don't know if my file comparison between those versions is valid for the changes to your queries.

mcrossley wrote:OK, found a copy of version 1.2!

The group by changes consist of where there is a function applied to the date

mcrossley wrote:Looks like those changes have been made by the owner of that site.

sfws wrote: Aha, that is the insight I needed - Group By needs to be applied to the query output, not the input column!
I know there is some reference to the GROUP BY problem earlier in the thread, but it does not explain that subtle detail.
Thank you, Mark.

sfws wrote:

mcrossley wrote:Looks like those changes have been made by the owner of that site.

I could ask him, but a) he does not have any knowledge of PHP, and b) he is in Northern hemisphere, so I am sure he will deny all. I may be wrong, but your posting of 30 Oct 2016 in this thread does suggest that you were considering the frost query logic, so I still suspect it was a change you made, and subsequently lost.

Mrc wrote:Ok so here's the files.

mcrossley wrote:Meanwhile can you send me a link to that source

Mrc wrote:Well, finally i got it working also Atleast almost, the max temperatures are wrong. There is data from 2015, 2016 and 2017 but there should be data only from this year because last month was so hot. Any idea what's wrong?