Pre release tests (1.6.7 R.C)

User avatar
axew3
w3all User
w3all User
Posts: 2883
Joined: Fri Jan 22, 2016 5:15 pm
Location: Italy
Contact:

Pre release tests (1.6.7 R.C)

Post by axew3 »

To test code before release:
https://plugins.trac.wordpress.org/brow ... tion/trunk

Code: Select all

= 1.6.7 =
*Release Date - ?? Apr, 2017*

* Add profile language update between phpBB and WordPress for users
* Fix deactivated phpBB user re-added in WordPress (maybe deactivated in phpBB due to deletion of the user in WordPress). Now a deactivated phpBB user will not be added or re-added into WordPress, as default behavior
* Fix display correct number of topics for each different widget instance setup
* Add avatar row on wp login widget user's info
* Fix Last Topics Widget html output to be more like any WP widget default output: fix correct separation between code and text output on same file (views/phpbb_last_topics.php)
* Improve query execution time for Last Topics Widget when Avatar active: the query is executed by searching for 'phpBB email hash' which is Key Index into phpBB db, and not by username.
* Fix uninitialized variables bug in WP_w3all admin config page, that was leading WP_w3all admin config page to emit notice errors as html output into Config fields about avatars or Preferences (where php ini were set to display php errors/notices: it was breaking options fields).
Unfortunately, i can't find out where the pre-release should be downloaded as unique zip, so any file need to be downloaded.
User avatar
axew3
w3all User
w3all User
Posts: 2883
Joined: Fri Jan 22, 2016 5:15 pm
Location: Italy
Contact:

Re: Pre release tests (1.6.7 R.C)

Post by axew3 »

To test code before release:
https://plugins.trac.wordpress.org/brow ... tion/trunk

On the above going on test code, many files have been patched and changed due to all above fixes.
views/phpbb_last_topics.php have change code, wrapping <table> elements inside default WordPress elements <ul> <li>, for both avatar or not avatar output. Little css style cleanup as suggested by @kaspir for table code (to review)

The code has been prepared for the addition about 'css classes' to elements (option that will be added into wp_w3all config page), so on plugin update will be not necessary re-apply css styles on widget, if it were customized.

files
views/login_form_include_iframe_mode_links.php
and
views/login_form_include_noiframe_mode_links.php
have been also updated to add avatar and to fix html output as above (separating the html code from text for example)
the code that display avatar on code

Code: Select all

echo '<li class="">' . get_avatar(get_current_user_id(), $w3all_last_t_avatar_dim) . '</li>';
has been set by default as is the dimension that is setup for avatars on last topics widget.
Maybe this need to be a separated option on config?

.....
Goofkop
User www
User www
Posts: 63
Joined: Sun Feb 26, 2017 8:33 am

Re: Pre release tests (1.6.7 R.C)

Post by Goofkop »

So i am back from Finland and see a new update.

I updated the plugin but in the login widget i didn't see my forum avatar.

http://imgur.com/a/CkKpK

And i have the option: Use phpBB avatar to replace WordPress user's avatar on Yes.

Before the update i saw my forum avatar on my replies and other posts. But now it is the unknown avatar i can choose on my WP options.
User avatar
kaspir
Moderator
Moderator
Posts: 88
Joined: Mon Mar 20, 2017 2:38 pm
Location: USA
Contact:

Re: Pre release tests (1.6.7 R.C)

Post by kaspir »

Doing this now:
-Saving/backing up my current plugin files
-Updating to new version
-Re-Applying my custom edits

Will be back to inform, maybe get you something about that avatar @Goofkop

EDIT: Goofkop is correct, on plugin update, before doing any changes, the avatar is only pulling the gravatar links only.
Image
World of Phaos RPG online is making it's come back! Play free now!
Check out phpBB contributions & extension downloads. :P
User avatar
axew3
w3all User
w3all User
Posts: 2883
Joined: Fri Jan 22, 2016 5:15 pm
Location: Italy
Contact:

Re: Pre release tests (1.6.7 R.C)

Post by axew3 »

checking, because the code search for email hashes now, and so something is wrong ... arghhh
User avatar
kaspir
Moderator
Moderator
Posts: 88
Joined: Mon Mar 20, 2017 2:38 pm
Location: USA
Contact:

Re: Pre release tests (1.6.7 R.C)

Post by kaspir »

axew3 wrote: Sat Apr 01, 2017 10:50 am checking, because the code search for email hashes now, and so something is wrong ... arghhh

YO, the problem is in this file: class.wp.w3all-phpbb.php

I replaced just this one file with the same file from version 1.6.6 and all avatars popped up! Everyone use this as a temporary fix until the boss submits patch fix. ;)
class.wp.w3all-phpbb_1.6.6.zip
Attached if you accidentally misplaced thru plugin update.
(15.07 KiB) Downloaded 203 times
EDIT: I think using the 1.6.6 file here made my 'unread post' marker not work now. :( But it's a suitable resolution for me for now.
Image
World of Phaos RPG online is making it's come back! Play free now!
Check out phpBB contributions & extension downloads. :P
Post Reply