Attila 30 Posted October 17, 2018 Report Share Posted October 17, 2018 Hi folks. By now you would be aware that Ver 7 upgrade has some unintended consequences, I thought that placing all know effects under one topic will be helpful, please add your findings herein below, I have alerted X2Josef 1) Email "Admin.maxFileSize" - Error "Admin.maxFileSize" is not defined. /home/x2/public_html/x2engine/framework/base/CComponent.php L130 edit protected/components/behaviors/EmailDeliveryBehavior.php Change -> if($size > ($admin->maxFileSize * 1024 * 1024)) { // 10mb file size limit is default To -> if($size > (10 * 1024 * 1024)) { // 10mb file size limit 2) Database fields: - Contacts Module - renames field "Accounts" (name) to "Company" (company) and creates another "Accounts" (name) field which is empty there may be consequences if you have workflows or standard documentation with dependencies on Accounts (name) fields in the Contacts module. 1 Quote Link to post Share on other sites
eugeneb 50 Posted October 17, 2018 Report Share Posted October 17, 2018 Agree with Attila on those points. Weird to add Company as another linked field. I would have just named the field in the form and kept accounts. Here are other bugs following your numbering sequence: 3) Reports module left menu LISTS option creates Error 500. Not sure what this Lists option does, assuming its to do reports on lists? 4.a ) eMail from Contact records creates over-sized dialogue box hiding the Top bar of the email dialogue box. Thus to resize in order to type email means zooming out of your browser to see the complete email dialogue box. then resizing it. 4.b ) Resizing the bottom and right of the email dialogue box has some hidden gap. Hovering over the box border doesn't show the sizing handle, you have to move away from the actual box to get the sizing handle. Minor but will confuse many users. 1 Quote Link to post Share on other sites
X2Peter 45 Posted October 17, 2018 Report Share Posted October 17, 2018 Hello All, Thank you for the feedback! We will be releasing a hotfix sometime next week in order to address the 'maxFileSize' issue (thank you Atilla for providing a workaround/fix) as well as the email UI bugs. The name change from 'Accounts' to 'Company' was added in order to add a proper conversion from a 'Contact' record to an 'Account' record and vice-versa. This change was in the attribute label only so the workflows using the field should still work. However we will address this issue in the hotfix as well in order to keep custom workflows compatible with the changes. We are currently in the process of updating the Github project. The new changelog will be included in the update. Let me know if you have any questions/concerns. 1 Quote Link to post Share on other sites
Attila 30 Posted October 18, 2018 Author Report Share Posted October 18, 2018 Thanks Peter, Just check that field "'Accounts' to 'Company' This change was in the attribute label only" believe me it was more than that, definitely field = company label = Company. as I describe above Quote Link to post Share on other sites
eugeneb 50 Posted October 18, 2018 Report Share Posted October 18, 2018 Thanks Peter. Quote Link to post Share on other sites
eugeneb 50 Posted October 19, 2018 Report Share Posted October 19, 2018 More bugs... 5. Lists in the other modules seem to be problematic. Examples5.a) list available in left menu of a module isn't available in the module top menu drop-down. Inconsistency. 5. Reports Lists menu option crashes 5.c) Products Lists menu option crashes 5.d) Leads Lists5.d.1) Menu option changes left menu and then doesnt have Leads List as top option. Selecting Lists from Leads becomes a Create menu feature. Should have Leads still at the top5.d.2)) Leads Lists - Creating dynamic list criteria comes from field in Leads module. However, grid view of the list shows fields from Contacts module, not Leads module. This Gridview cant really be used with your fields from Leads module. 5.e) Opportunity Lists - Creating dynamic list criteria comes from field in Opportunities module. However, grid view of the list shows fields from Contacts module, not Oppertunties module. This Gridview cant really be used with your fields from Opps. 5.f) Service Cases Lists menu option crashes Hope this helps 1 Quote Link to post Share on other sites
JacobD 3 Posted October 19, 2018 Report Share Posted October 19, 2018 5.e) Opportunity Lists - Creating dynamic list criteria comes from field in Opportunities module. However, grid view of the list shows fields from Contacts module, not Oppertunties module. This Gridview cant really be used with your fields from Opps. +1 On this. I was about to post the same issue. 1 Quote Link to post Share on other sites
DonnaParrish 5 Posted October 23, 2018 Report Share Posted October 23, 2018 My update failed because there was a CRC error on a php file (quote.php) that we had changed because we needed it to say different things for our quotes. Evidently, the upgrade now does a crc check. No problem. I tried to roll back the update. I got a 500 error. No problem, I reverted to the old files and old data. Unfortunately, my 500 error has stuck, so my revert was not as complete as I had hoped. Now what??? Error Code: 500Error Message: Function mcrypt_decrypt() is deprecatedStack Trace: [click to toggle display]X2Engine Version: 6.9.3PHP Version: 7.1.23 I posted this on http://community.x2crm.com/topic/3712-upgrade-failed-then-going-back-to-old-version-fails/before I saw this thread. Are there any suggestions for recovery? We are presently D.O.A. Quote Link to post Share on other sites
Attila 30 Posted October 24, 2018 Author Report Share Posted October 24, 2018 Hi Donna, there was a database field in Contacts that was changed that most likely caused the 500 error, I always backup the domain so that I can roll back with a known working instance if needed. I'm sorry I don't have a simple fix at my fingertips Attila Quote Link to post Share on other sites
DonnaParrish 5 Posted October 24, 2018 Report Share Posted October 24, 2018 Thanks, Attila! we have rolled our php version back and now the old version is working. 2 Quote Link to post Share on other sites
X2Peter 45 Posted October 29, 2018 Report Share Posted October 29, 2018 Hello Donna, X2CRM is not compatible with PHP 7.1 at this time. We will reach full compatibility before Mid-December. We do recommend backing up any site files as well as the database before update. 4 Quote Link to post Share on other sites
Attila 30 Posted November 12, 2018 Author Report Share Posted November 12, 2018 Another bug detected: When running a campaign triggered on a future date/time: The campaign runs and sends emails as it should, but there in no report of what was delivered, opened and clicked though... Conversely if a campaign is run right away the reports work as they should Attila Quote Link to post Share on other sites
Craig Donovan 1 Posted December 9, 2018 Report Share Posted December 9, 2018 Hello Donna, X2CRM is not compatible with PHP 7.1 at this time. We will reach full compatibility before Mid-December. We do recommend backing up any site files as well as the database before update.HiAny news on the compatibility of X2CRM with PHP 7.1 or 7.2? Craig Quote Link to post Share on other sites
ylandsman 3 Posted March 19, 2019 Report Share Posted March 19, 2019 Using 7.1 and php 7.3 I got a 500 error from framework/i18n/CNumberFormatter.php when it was passed a non numeric value. In previous versions of php, this resulted in the return of 0. To alleviate this, I added a check for numeric at the beginning of the function. If I make changes like this should I submit them to the community github? 2 Quote Link to post Share on other sites
Attila 30 Posted March 20, 2019 Author Report Share Posted March 20, 2019 Using 7.1 and php 7.3 I got a 500 error from framework/i18n/CNumberFormatter.php when it was passed a non numeric value. In previous versions of php, this resulted in the return of 0. To alleviate this, I added a check for numeric at the beginning of the function. If I make changes like this should I submit them to the community github? Thanks Ylandsman, I think you should do both as many members on this forum can benefit from your findings .... 1 Quote Link to post Share on other sites
jack 29 Posted March 27, 2019 Report Share Posted March 27, 2019 Thanks for the find Ylandsman, can you please also post your detailed fix here so we can be sure to added to our next update. -Jack Quote Link to post Share on other sites
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.