Почему не срабатывает transient api?

Question

[m325]

Доброго времени суток, уважаемые гуру!
Беру погоду из xml. Чтобы не делать запросы при каждом обновление страницы.
Нашел решение это сделать через transient api для wordpress. Чтобы запрос обновлялся сам раз в час. Подскажите, пожалуйста что не так. Не работает

function new_weather () {

    // Выдача из транзитного кэша
    $cached = get_transient( 'weather_cache' );
    if ( $cached == false ) {
        return $cached;

    $result = simplexml_load_file( 'http://export.yandex.ru/weather-ng/forecasts/27612.xml' );
    $temperature = $result->fact->temperature;
    

    // Запись в транзитный кэш на 1 час
    set_transient( 'weather_cache', $temperature, 1 * HOUR_IN_SECONDS );

    return $temperature;
    
}
}

Comments

[Алексей Уколов]

Я в вордпрессе, конечно, не разбираюсь, может, у него такая магия, но функция наверняка должна что-то возвращать, а переменная $cache вообще нигде не используется.

Answer 1

[WP Panda]

Учитесь читать код, вопросов такого плана не будет.

function new_weather () {

    // Выдача из транзитного кэша
    $cached = get_transient( 'weather_cache' );
    if ( $cached !== false ) 
        return $cached;

    $result = simplexml_load_file( 'http://export.yandex.ru/weather-ng/forecasts/27612.xml' );
    $temperature = $result->fact->temperature;
    
    // Запись в транзитный кэш на 1 час
    set_transient( 'weather_cache', $temperature, 1 * HOUR_IN_SECONDS );

    return $temperature;
   
}

Comments

[m325]

?

[WP Panda]

m325: сравните мой код и ваш, мой правильный ваш ничего не вернет

[m325]

WP Panda: Вы правы. Извиняюсь за не внимательность.

[m325]

WP Panda: функция срабатывает но не выводит результат

[WP Panda]

стесняюсь спросить... а каким образом вы его выводите?

[m325]

WP Panda: в cайдбаре <?php echo new_weather(); ?> это не правильно ?

[WP Panda]

set_transient( 'weather_cache', (string)$temperature, 1 * HOUR_IN_SECONDS );

сделайте так

[m325]

WP Panda: Спасибо, что помогаете разобраться. Заменил, так как вы написали. Не помогло.

[WP Panda]

это от того что у вас уже переменная записана
закомментируйте проверку кэша так

/* $cached = get_transient( 'weather_cache' );
    if ( $cached !== false ) 
        return $cached; */

после этого обновите страницу,
затем верните обратно и будет счастье

[m325]

WP Panda: cпасибо, счастье есть :) подписался на ваш блог буду прокачивать скилл. еще раз огромное спасибо