Jump to content
X2Community Forums

Known Issues with ver 7 upgrade


Recommended Posts

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. 

  • Like 1
Link to post
Share on other sites

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.

  • Like 1
Link to post
Share on other sites

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.

  • Like 1
Link to post
Share on other sites

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 

Link to post
Share on other sites

More bugs...

 

5. Lists in the other modules seem to be problematic. Examples

5.a) list available in left menu of a module isn't available in the module top menu drop-down. Inconsistency.

 

5.B) Reports Lists menu option crashes

 

5.c) Products Lists menu option crashes

 

5.d) Leads Lists
5.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 top

5.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

  • Like 1
Link to post
Share on other sites

 

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.

  • Like 1
Link to post
Share on other sites

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: 500
Error Message: Function mcrypt_decrypt() is deprecated
Stack Trace: [click to toggle display]
X2Engine Version: 6.9.3
PHP 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.

Link to post
Share on other sites

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 

Link to post
Share on other sites
  • 2 weeks later...

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       

Link to post
Share on other sites
  • 4 weeks later...
  • 3 months later...

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?

  • Like 2
Link to post
Share on other sites

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 .... 

  • Like 1
Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
×
×
  • Create New...