-
5.0.3 TypeError: IPS\gallery\Image::get__dimensions(): Return value must be of type array
Was mentioned to be fixed few releases ago but still seeing these errors show up. Full stacktrace: TypeError: IPS\gallery\Image::get__dimensions(): Return value must be of type array, null returned in /home/domains/public_html/community/applications/gallery/sources/Image/Image.php:292 Stack trace: #0 /home/domains/public_html/community/system/Patterns/ActiveRecord.php(431): IPS\gallery\Image->get__dimensions() #1 /home/domains/public_html/community/static/templates/gallery_front_browse.php(6401): IPS\Patterns\ActiveRecord->__isset() #2 /home/domains/public_html/community/system/Theme/SandboxedTemplate.php(68): IPS\Theme\class_gallery_front_browse->tableRowsThumbs() #3 /home/domains/public_html/community/static/templates/gallery_front_browse.php(3934): IPS\Theme\SandboxedTemplate->__call() #4 /home/domains/public_html/community/system/Theme/SandboxedTemplate.php(68): IPS\Theme\class_gallery_front_browse->imageTable() #5 /home/domains/public_html/community/system/Helpers/Table/Table.php(606): IPS\Theme\SandboxedTemplate->__call() #6 /home/domains/public_html/community/applications/gallery/modules/front/gallery/browse.php(391): IPS\Helpers\Table\Table->__toString() #7 /home/domains/public_html/community/applications/gallery/modules/front/gallery/browse.php(147): IPS\gallery\modules\front\gallery\browse->_category() #8 /home/domains/public_html/community/system/Dispatcher/Controller.php(139): IPS\gallery\modules\front\gallery\browse->manage() #9 /home/domains/public_html/community/system/Content/Controller.php(124): IPS\Dispatcher\Controller->execute() #10 /home/domains/public_html/community/applications/gallery/modules/front/gallery/browse.php(81): IPS\Content\Controller->execute() #11 /home/domains/public_html/community/system/Dispatcher/Dispatcher.php(169): IPS\gallery\modules\front\gallery\browse->execute() #12 /home/domains/public_html/community/index.php(16): IPS\Dispatcher->run() #13 {main} Backtrace: #0 /home/domains/public_html/community/system/Theme/SandboxedTemplate.php(78): IPS\Log::log() #1 /home/domains/public_html/community/static/templates/gallery_front_browse.php(3934): IPS\Theme\SandboxedTemplate->__call() #2 /home/domains/public_html/community/system/Theme/SandboxedTemplate.php(68): IPS\Theme\class_gallery_front_browse->imageTable() #3 /home/domains/public_html/community/system/Helpers/Table/Table.php(606): IPS\Theme\SandboxedTemplate->__call() #4 /home/domains/public_html/community/applications/gallery/modules/front/gallery/browse.php(391): IPS\Helpers\Table\Table->__toString() #5 /home/domains/public_html/community/applications/gallery/modules/front/gallery/browse.php(147): IPS\gallery\modules\front\gallery\browse->_category() #6 /home/domains/public_html/community/system/Dispatcher/Controller.php(139): IPS\gallery\modules\front\gallery\browse->manage() #7 /home/domains/public_html/community/system/Content/Controller.php(124): IPS\Dispatcher\Controller->execute() #8 /home/domains/public_html/community/applications/gallery/modules/front/gallery/browse.php(81): IPS\Content\Controller->execute() #9 /home/domains/public_html/community/system/Dispatcher/Dispatcher.php(169): IPS\gallery\modules\front\gallery\browse->execute() #10 /home/domains/public_html/community/index.php(16): IPS\Dispatcher->run() #11 {main}And the error on the page itself:
-
Sander Vancanneyt started following 5.x no styling for word expansion feature (abbr element)
-
5.x no styling for word expansion feature (abbr element)
Using the word expansion feature is something we heavily use for many acronyms in our community. However in Invision Community 5 there is no styling of the <abbr> element that is generated for the word expansions. As the styling is missing, users don't even see that you can have a hover on desktop explaining what the abbreviation is. So that I consider a bug as the styling is missing. in v4 there was styling applied to the abbr element. Second, for mobile users you can't hover on an <abbr> element so for our mobile users they don't get to see what the word expansion is. a solution to make not work for mobile users can be found on this url: https://bitsofco.de/making-abbr-work-for-touchscreen-keyboard-mouse/ Conclusion: make sure word expansions are visible in the topics so desktop users can hover to know what the abbreviation stands for. Second make it also work for mobile users.
-
Gallery php error
In the gallery, a php error shows up when viewing a category. Here’s the full error from the log: TypeError: IPS\gallery\Image::get__dimensions(): Return value must be of type array, null returned in /home/domains/.com/public_html/community/applications/gallery/sources/Image/Image.php:292 Stack trace: #0 /home/domains/.com/public_html/community/system/Patterns/ActiveRecord.php(431): IPS\gallery\Image->get__dimensions() #1 /home/domains/.com/public_html/community/static/templates/gallery_front_browse.php(6401): IPS\Patterns\ActiveRecord->__isset() #2 /home/domains/.com/public_html/community/system/Theme/SandboxedTemplate.php(68): IPS\Theme\class_gallery_front_browse->tableRowsThumbs() #3 /home/domains/.com/public_html/community/static/templates/gallery_front_browse.php(3934): IPS\Theme\SandboxedTemplate->__call() #4 /home/domains/.com/public_html/community/system/Theme/SandboxedTemplate.php(68): IPS\Theme\class_gallery_front_browse->imageTable() #5 /home/domains/.com/public_html/community/system/Helpers/Table/Table.php(606): IPS\Theme\SandboxedTemplate->__call() #6 /home/domains/.com/public_html/community/applications/gallery/modules/front/gallery/browse.php(391): IPS\Helpers\Table\Table->__toString() #7 /home/domains/.com/public_html/community/applications/gallery/modules/front/gallery/browse.php(147): IPS\gallery\modules\front\gallery\browse->_category() #8 /home/domains/.com/public_html/community/system/Dispatcher/Controller.php(139): IPS\gallery\modules\front\gallery\browse->manage() #9 /home/domains/.com/public_html/community/system/Content/Controller.php(124): IPS\Dispatcher\Controller->execute() #10 /home/domains/.com/public_html/community/applications/gallery/modules/front/gallery/browse.php(81): IPS\Content\Controller->execute() #11 /home/domains/.com/public_html/community/system/Dispatcher/Dispatcher.php(169): IPS\gallery\modules\front\gallery\browse->execute() #12 /home/domains/.com/public_html/community/index.php(16): IPS\Dispatcher->run() #13 {main}
-
Word expansion for mobile users
When using word expansion, it's only useful for desktop users as they hover the word that the expansion is visible. But that's not the case when you're on a mobile device. Tapping the word that should be expanded is just not showing. So the feature only works on desktop which is a shame for all the mobile users who wanna know what the abbreviation is of a word. Also in version 5 it's not easily visible that the word has en expansion. In v4 you had a dotted line under the word so you'd know you could hold your mouse on it to reveal the expansion, and the mouse cursor changed to a question mark but that's not the case anymore too in v5. (there is no styling for the abbr html element in v5) A possible solution to make it work for mobile devices can be found here: https://bitsofco.de/making-abbr-work-for-touchscreen-keyboard-mouse/ Hope Invision will incorporate this in next release.
-
v5 questions
Ok, so when using a hook, syntax to check user groups that should get ads would be (documentation lacks a bit what the member object contains): {{if !in_array(member.id.primaryGroup.id,[4,9,17,282] }} Ad code here {{endif}}
-
-
v5 questions
As many use Adsense or other ad providers to monetise their forums we do need a solution to place the JavaScript ad code into the head section and based upon weather user is logged in or not or has bought a subscription (nexus) to remove the ads. the solution as it is in v4 where a custom location could be set for the ad script and including it in the globalTemplate head section was a good solution. As this is missing in v5 we definitely need a valid alternative. Weather it’s a hook that ads the script to the head section or something else, we would need a solution.
-
Cookie Notice
Yes, but the main message text is only available in one language, it isn't available in the translation manager. Which should be a more convenient place for it. And also, after upgrading to the latest version, the translated text in the translation manager is deleted by the upgrade proces, making it an extra work to re-translate the strings to the previous state.
-
Cookie Notice
For multi-lingual sites it is not very practical. You can change the text in only one language, the text is not translatable in multiple languages. The 'Got it' button is the same, you can't even change the text (editing the XML would work, but for noob users it would be much better if those frontend texts where also available for translation in the translation manager)