Replace deprecated create_function().

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Rangaswini created an issue. See original summary.

Rangaswini’s picture

Priority: Normal » Critical
Status: Active » Needs review
FileSize
2.5 KB
Rangaswini’s picture

Status: Needs review » Needs work
Rangaswini’s picture

Title: Replace deprecated function. » Replace eval function.
Status: Needs work » Needs review
FileSize
4.02 KB

I have removed deprecated function eval(). Added external library eval-math-master to evaluate the mathematical expression.

Rangaswini’s picture

Assigned: Rangaswini » Unassigned
truptidiwani’s picture

Assigned: Unassigned » truptidiwani
truptidiwani’s picture

Status: Needs review » Reviewed & tested by the community

I have reviewed the patch-
https://www.drupal.org/files/issues/2019-10-04/remove_deprecated_functio... and thus its working well.

Checked:
- The patch is being applied successfully and have tested the following test cases.- For addition, subtraction,divide,multiply,backsapce,clear.
- The libarary eval-math is applied and is working well.

truptidiwani’s picture

Assigned: truptidiwani » Unassigned

JayKandari’s picture

Status: Reviewed & tested by the community » Fixed

Thank you @Rangaswini & @truptidiwani for your contributions.
Patch looks okay to me.
There are few coding standards issue in this, but this can be covered in #3086737 ticket.

JayKandari’s picture

JayKandari’s picture

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.