Jump to content


Replying to Bug In Ithemes-Exchange/api/pages.php


Post Options

  or Cancel


Topic Summary

WildPlatypus

Posted 21 May 2014 - 03:16 PM

OK - thanks - then you can ignore my simul-post above.  I think it's in everyone's best interest (to fix the changelog)! ;-)

WildPlatypus

Posted 21 May 2014 - 03:16 PM

You're welcome.

With all due respect, whether or not the readme.txt validates in its current state is not the issue.  It is missing an important section. And, the == Changelog == section is required if you are hosting the plugin on wordpress.org and expect the update link on the plugin update page of the user's Wordpress dashboard to be able to pull up the changelog info/tab.

A separate, public changelog at your site is nice, but not helpful to users at the moment of upgrade, since it is not tied to the WP plugin update interface at the point when users are going to update their plugin.

Please re-think the approach on this issue.  It should just work when the user clicks the link that's already there. It's common sense and optimal in terms of UX.

http://wordpress.org/news/2009/07/improving-your-plugin-changelogs/
http://wordpress.org/plugins/about/readme.txt

Regards,

WP

Glenn Ansley

Posted 21 May 2014 - 03:15 PM

With that being said, i've updated the readme for you.

Glenn Ansley

Posted 21 May 2014 - 02:39 PM

Thanks for the heads up on the misspelled word. The == Changelog == section isn't required by WordPress. Our readme.txt validates on the .org hosted validator.
We have a public changelog available here: http://ithemes.com/codex/page/Exchange_Changelog and it is linked in the update notification part of our readme.txt

Ronald

Posted 21 May 2014 - 02:36 PM

Hi,

I will pass it on to the developers, thanks,

Ronald

WildPlatypus

Posted 21 May 2014 - 12:30 PM

There is a bug in /api/pages.php

do_action( 'it_exchnage_register_page', $page, $options );

exchange is spelled incorrectly as exchnage which will likely prevent the hook from being correctly executed.

This bug exists in the current version as of today.  (1.8.2)

I found this bug in the process of trying to figure out why all iThemes plugins fail to correctly display their changelogs from the plugin update window.  

Further (and as reported in my other thread), Exchange shows a Product Description instead of the changelog when clicking on the changelog link, which likely points to the fact that the wordpress readme.txt file being hosted at wordpress.org either fails to include a valid changelog section or is otherwise formatted incorrectly.

Based on a review of the readme.txt included locally with the plugin, it appears to be the former, since there is no == Changelog == section in the readme.txt.  Please follow the Wordpress standards so that users can benefit from being able to see the changelog before updating their plugins.

Please pass this feedback directly to engineering.  Thx.

Review the complete topic (launches new window)