search

lafi-business-group / lafi-sa.issues

Issue only repo for public. Please report all issues of lafi-sa.com in this repo
https://www.lafi-sa.com
0 stars 0 forks source link

Bulk pre-order template upload error #3

Closed majeedpalakkal closed 4 years ago

majeedpalakkal commented 4 years ago

Excel template doesn't upload, it's getting an error. Error message shown below

Fatal error: 'break' not in the 'loop' or 'switch' context in /home2/majeedlafi/public_html/libraries/excel/PHPExcel/Calculation/Functions.php on line 574

azharakbar commented 4 years ago

@majeedpalakkal Pls adhere to the template of NEW ISSUE

Reproduction steps not found Reproduction steps also include attaching the excel file that produces the error

Pls mention whether reproducing or not in testServer

Follow all these steps in all future issues....Use the template for BUG REPORT to add a new issue

majeedpalakkal commented 4 years ago

Panasonic.xlsx

Reproduction steps

  1. https://www.lafi-sa.com/new_test/manage
  2. Click "Bulk Pre order entry"
  3. Choose the excel template
  4. Upload

New_test has the same problem

azharakbar commented 4 years ago

Duplicate of #4

majeedpalakkal commented 4 years ago

Yes, It's working fine in New_test now. You can merge

majeedpalakkal commented 4 years ago

The bulk pre-order template is able to upload, but it doesn't display in front end

azharakbar commented 4 years ago

Ok

azharakbar commented 4 years ago

preOrder feature was not merged properly

majeedpalakkal commented 4 years ago

Now the pre-order items displaying and working properly.

However, there is another problem related to this. When the Pre-order items & special offer items active on same time, both together doesn't display in front-end. Please check

azharakbar commented 4 years ago

@majeedpalakkal pls give more details with screenshots.. Expected screenshots.. And what is the currently displaying screenshots

majeedpalakkal commented 4 years ago

Really Sorry, it was reported by a mistake by my side. The solution is OK and working properly.

majeedpalakkal commented 4 years ago

Please merge it to mainline