Kill overloaded cast operators in CJS_Value.

fpdfsdk/src/javascript/PublicMethods.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
  
 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
 
 #include "../../include/javascript/JavaScript.h"
 #include "../../include/javascript/IJavaScript.h"
 #include "../../include/javascript/JS_Define.h"
 #include "../../include/javascript/JS_Object.h"
@@ skipping 412 matching lines @@
 }
 
 CJS_Array CJS_PublicMethods::AF_MakeArrayFromList(v8::Isolate* isolate, CJS_Value val)
 {
         CJS_Array StrArray(isolate);
         if(val.IsArrayObject())
         {
                 val.ConvertToArray(StrArray);
                 return StrArray;
         }
-        CFX_WideString wsStr = val.operator CFX_WideString();
+        CFX_WideString wsStr = val.ToCFXWideString();
         CFX_ByteString t = CFX_ByteString::FromUnicode(wsStr);
         const char * p = (const char *)t;
 
 
         int ch = ',' ;
         int nIndex = 0;
 
         while (*p)
         {
                 const char * pTemp = strchr(p, ch);
@@ skipping 620 matching lines @@
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
         if(!pEvent->m_pValue)
                 return FALSE;
         CFX_WideString& Value = pEvent->Value();        
         CFX_ByteString strValue = StrTrim(CFX_ByteString::FromUnicode(Value));
         
         if (strValue.IsEmpty()) return TRUE;
         
-        int iDec = params[0];
-        int iSepStyle = params[1];
-        int iNegStyle = params[2];
+        int iDec = params[0].ToInt();
+        int iSepStyle = params[1].ToInt();
+        int iNegStyle = params[2].ToInt();
         // params[3] is iCurrStyle, it's not used.
-        std::wstring wstrCurrency(params[4].operator CFX_WideString());
-        FX_BOOL bCurrencyPrepend = params[5];
+        std::wstring wstrCurrency(params[4].ToCFXWideString());
+        FX_BOOL bCurrencyPrepend = params[5].ToBool();
         
         if (iDec < 0) iDec = -iDec;
         
         if (iSepStyle < 0 || iSepStyle > 3)
                 iSepStyle = 0;
         
         if (iNegStyle < 0 || iNegStyle > 3)
                 iNegStyle = 0;
         
         
@@ skipping 171 matching lines @@
 //function AFNumber_Keystroke(nDec, sepStyle, negStyle, currStyle, strCurrency, bCurrencyPrepend)
 FX_BOOL CJS_PublicMethods::AFNumber_Keystroke(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
         CJS_EventHandler* pEvent = pContext->GetEventHandler();
         ASSERT(pEvent != NULL);
         
         if(params.size() < 2)
                 return FALSE;
-        int iSepStyle = params[1];
+        int iSepStyle = params[1].ToInt();
         
         if (iSepStyle < 0 || iSepStyle > 3)
                 iSepStyle = 0;
         if(!pEvent->m_pValue)
                 return FALSE;
         CFX_WideString & val = pEvent->Value(); 
         CFX_WideString & w_strChange = pEvent->Change();
     CFX_WideString w_strValue = val;
 
         if (pEvent->WillCommit())
@@ skipping 117 matching lines @@
         CJS_EventHandler* pEvent = pContext->GetEventHandler();
         ASSERT(pEvent != NULL);
 
     if (params.size() != 2)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
         if(!pEvent->m_pValue)
                 return FALSE;
+
         CFX_WideString& Value = pEvent->Value();
-        
-//     HWND hMainFrame = NULL;
-//      
-//      CPDFSDK_FormFillApp *pApp = pContext->GetReaderApp();
-//      ASSERT(pApp);
-//      hMainFrame = pApp->GetMainFrameWnd();
-                
         CFX_ByteString strValue = StrTrim(CFX_ByteString::FromUnicode(Value));
-        
         if (strValue.IsEmpty())
                 return TRUE;
-        
-        int iDec = params[0];
-        int iSepStyle = params[1];
-        
-        //ASSERT(iDec > 0);
+
+        int iDec = params[0].ToInt();
         if (iDec < 0)
                 iDec = -iDec;
-        
+
+        int iSepStyle = params[1].ToInt();
         if (iSepStyle < 0 || iSepStyle > 3)
                 iSepStyle = 0;
-        
-        
+
         //////////////////////////////////////////////////////
         //for processing decimal places
         double dValue = atof(strValue);
         dValue *= 100;
         if (iDec > 0)
                 dValue += DOUBLE_CORRECT;//УÕý
 
         int iDec2;
         FX_BOOL bNagative = FALSE;
         strValue = fcvt(dValue,iDec,&iDec2,&bNagative);
@@ skipping 77 matching lines @@
 {
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
         CJS_EventHandler* pEvent = pContext->GetEventHandler();
         ASSERT(pEvent != NULL);
 
         if (params.size() != 1)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
-        }       
+        }
         if(!pEvent->m_pValue)
                 return FALSE;
+
         CFX_WideString& val = pEvent->Value();
-        
-        CFX_WideString strValue = val;  
-        if (strValue.IsEmpty()) return TRUE;            
+        CFX_WideString strValue = val;
+        if (strValue.IsEmpty())
+                return TRUE;
 
-        CFX_WideString sFormat = params[0].operator CFX_WideString();
-
+        CFX_WideString sFormat = params[0].ToCFXWideString();
         FX_BOOL bWrongFormat = FALSE;
         double dDate = 0.0f;
 
         if(strValue.Find(L"GMT") != -1)
         {
                 //for GMT format time
                 //such as "Tue Aug 11 14:24:16 GMT+08002009"
                 dDate = MakeInterDate(strValue);
         }
         else
         {
                 dDate = MakeRegularDate(strValue,sFormat,bWrongFormat);
         }
 
         if (JS_PortIsNan(dDate))
         {
                 CFX_WideString swMsg;
                 swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSPARSEDATE), sFormat.c_str());
                 Alert(pContext, swMsg);
                 return FALSE;
         }
-        
+
         val =  MakeFormatDate(dDate,sFormat);
-
         return TRUE;
 }
 
 double CJS_PublicMethods::MakeInterDate(CFX_WideString strValue)
 {
         int nHour;
         int nMin;
         int nSec;
         int nYear;
         int nMonth;
@@ skipping 55 matching lines @@
 {
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
         CJS_EventHandler* pEvent = pContext->GetEventHandler();
         ASSERT(pEvent != NULL);
 
         if (params.size() != 1)
         {
                 sError = L"AFDate_KeystrokeEx's parameters' size r not correct";
                 return FALSE;
-        }       
-        
+        }
+
         if (pEvent->WillCommit())
         {
                 if(!pEvent->m_pValue)
                         return FALSE;
                 CFX_WideString strValue = pEvent->Value();
-                if (strValue.IsEmpty()) return TRUE;
+                if (strValue.IsEmpty())
+                        return TRUE;
 
-                CFX_WideString sFormat = params[0].operator CFX_WideString();
-
+                CFX_WideString sFormat = params[0].ToCFXWideString();
                 FX_BOOL bWrongFormat = FALSE;
                 double dRet = MakeRegularDate(strValue,sFormat,bWrongFormat);
                 if (bWrongFormat || JS_PortIsNan(dRet))
                 {
                         CFX_WideString swMsg;
                         swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSPARSEDATE), sFormat.c_str());
                         Alert(pContext, swMsg);
                         pEvent->Rc() = FALSE;
                         return TRUE;
                 }
         }
         return TRUE;
 }
 
 FX_BOOL CJS_PublicMethods::AFDate_Format(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         v8::Isolate* isolate = ::GetIsolate(cc);
 
         if (params.size() != 1)
         {
                 CJS_Context* pContext = (CJS_Context*)cc;
                 ASSERT(pContext != NULL);
 
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
-        int iIndex = params[0];
+        int iIndex = params[0].ToInt();
         FX_LPCWSTR cFormats[] =  {L"m/d", L"m/d/yy", L"mm/dd/yy", L"mm/yy", L"d-mmm", L"d-mmm-yy", L"dd-mmm-yy",
                 L"yy-mm-dd", L"mmm-yy", L"mmmm-yy", L"mmm d, yyyy", L"mmmm d, yyyy",
                 L"m/d/yy h:MM tt", L"m/d/yy HH:MM" };
 
         ASSERT(iIndex < FX_ArraySize(cFormats));
 
         if (iIndex < 0)
                 iIndex = 0;
         if (iIndex >= FX_ArraySize(cFormats))
                 iIndex = 0;
@@ skipping 10 matching lines @@
 
         if (params.size() != 1)
         {
                 CJS_Context* pContext = (CJS_Context*)cc;
                 ASSERT(pContext != NULL);
 
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
-        int iIndex = params[0];
+        int iIndex = params[0].ToInt();
         FX_LPCWSTR cFormats[] =  {L"m/d", L"m/d/yy", L"mm/dd/yy", L"mm/yy", L"d-mmm", L"d-mmm-yy", L"dd-mmm-yy",
                 L"yy-mm-dd", L"mmm-yy", L"mmmm-yy", L"mmm d, yyyy", L"mmmm d, yyyy",
                 L"m/d/yy h:MM tt", L"m/d/yy HH:MM" };
 
         ASSERT(iIndex<FX_ArraySize(cFormats));
 
         if (iIndex < 0)
                 iIndex = 0;
         if (iIndex >= FX_ArraySize(cFormats))
                 iIndex = 0;
         CJS_Parameters newParams;
         CJS_Value val(isolate,cFormats[iIndex]);
         newParams.push_back(val);
         return AFDate_KeystrokeEx(cc,newParams,vRet,sError);
 }
 
 //function AFTime_Format(ptf)
 FX_BOOL CJS_PublicMethods::AFTime_Format(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         v8::Isolate* isolate = ::GetIsolate(cc);
 
         if (params.size() != 1)
         {
                 CJS_Context* pContext = (CJS_Context*)cc;
                 ASSERT(pContext != NULL);
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
-        int iIndex = params[0];
+        int iIndex = params[0].ToInt();
         FX_LPCWSTR cFormats[] = {L"HH:MM", L"h:MM tt", L"HH:MM:ss", L"h:MM:ss tt"};
 
         ASSERT(iIndex<FX_ArraySize(cFormats));
 
         if (iIndex < 0)
                 iIndex = 0;
         if (iIndex >= FX_ArraySize(cFormats))
                 iIndex = 0;
         CJS_Parameters newParams;
         CJS_Value val(isolate,cFormats[iIndex]);
         newParams.push_back(val);
         return AFDate_FormatEx(cc,newParams,vRet,sError);
 }
 
 FX_BOOL CJS_PublicMethods::AFTime_Keystroke(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         v8::Isolate* isolate = ::GetIsolate(cc);
         if (params.size() != 1)
         {
                 CJS_Context* pContext = (CJS_Context*)cc;
                 ASSERT(pContext != NULL);
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
-        int iIndex = params[0];
+        int iIndex = params[0].ToInt();
         FX_LPCWSTR cFormats[] = {L"HH:MM", L"h:MM tt", L"HH:MM:ss", L"h:MM:ss tt"};
 
         ASSERT(iIndex<FX_ArraySize(cFormats));
 
         if (iIndex < 0)
                 iIndex = 0;
         if (iIndex >= FX_ArraySize(cFormats))
                 iIndex = 0;
         CJS_Parameters newParams;
         CJS_Value val(isolate,cFormats[iIndex]);
@@ skipping 17 matching lines @@
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
 
         if (params.size() != 1)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
         std::string cFormat;
-        int iIndex = params[0];
+        int iIndex = params[0].ToInt();
 
         CJS_EventHandler* pEvent = pContext->GetEventHandler();
         ASSERT(pEvent != NULL);
 
         if(!pEvent->m_pValue)
                 return FALSE;
         CFX_WideString& Value = pEvent->Value();        
         std::string strSrc = CFX_ByteString::FromUnicode(Value).c_str();
         
         switch (iIndex) 
@@ skipping 38 matching lines @@
         if (params.size() < 1)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
         if(!pEvent->m_pValue)
                 return FALSE;
         CFX_WideString& valEvent = pEvent->Value();
 
-        CFX_WideString wstrMask = params[0].operator CFX_WideString();
-        if (wstrMask.IsEmpty()) return TRUE;
-        
+        CFX_WideString wstrMask = params[0].ToCFXWideString();
+        if (wstrMask.IsEmpty())
+                return TRUE;
+
         std::wstring wstrValue(valEvent);
-        
+
         if (pEvent->WillCommit())
         {
                 if (wstrValue.empty())
                         return TRUE;
                 int iIndexMask = 0;
                 for (std::wstring::iterator it = wstrValue.begin(); it != wstrValue.end(); it++)
                 {
                         wchar_t w_Value = *it;
-            if (!maskSatisfied(w_Value,wstrMask[iIndexMask]))
+                        if (!maskSatisfied(w_Value,wstrMask[iIndexMask]))
                                 break;
                         iIndexMask++;
                 }
 
                 if (iIndexMask != wstrMask.GetLength() || (iIndexMask != wstrValue.size() && wstrMask.GetLength() != 0))
                 {
                         Alert(pContext, JSGetStringFromID(pContext, IDS_STRING_JSAFNUMBER_KEYSTROKE));
                         pEvent->Rc() = FALSE;
                 }
                 return TRUE;
         }
 
-        
         CFX_WideString &wideChange = pEvent->Change();
         std::wstring wChange(wideChange);
-        
         if (wChange.empty())
                 return TRUE;
-    int iIndexMask = pEvent->SelStart();
-        //iIndexMask++;
-        
-        
+
+        int iIndexMask = pEvent->SelStart();
+
         if (wstrValue.length() - (pEvent->SelEnd()-pEvent->SelStart()) + wChange.length() > (FX_DWORD)wstrMask.GetLength())
         {
                 Alert(pContext, JSGetStringFromID(pContext, IDS_STRING_JSPARAM_TOOLONG));
                 pEvent->Rc() = FALSE;
                 return TRUE;
         }
-        
-        
+
         if (iIndexMask >= wstrMask.GetLength() && (!wChange.empty()))
         {
                 Alert(pContext, JSGetStringFromID(pContext, IDS_STRING_JSPARAM_TOOLONG));
                 pEvent->Rc() = FALSE;
                 return TRUE;
         }
-        
+
         for (std::wstring::iterator it = wChange.begin(); it != wChange.end(); it++)
         {
                 if (iIndexMask >= wstrMask.GetLength())
                 {
                         Alert(pContext, JSGetStringFromID(pContext, IDS_STRING_JSPARAM_TOOLONG));
                         pEvent->Rc() = FALSE;
                         return TRUE;
                 }
                 wchar_t w_Mask = wstrMask[iIndexMask];
                 if (!isReservedMaskChar(w_Mask))
                 {
-                        //wChange.insert(it,w_Mask);                            
                         *it = w_Mask;
                 }
                 wchar_t w_Change = *it;
-                
-        if (!maskSatisfied(w_Change,w_Mask))
+                if (!maskSatisfied(w_Change,w_Mask))
                 {
                         pEvent->Rc() = FALSE;
                         return TRUE;
                 }
                 iIndexMask++;
         }
-        
-        wideChange = wChange.c_str();   
-        
+
+        wideChange = wChange.c_str();
         return TRUE;
 }
 
 
 //function AFSpecial_Keystroke(psf)
 FX_BOOL CJS_PublicMethods::AFSpecial_Keystroke(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         v8::Isolate* isolate = ::GetIsolate(cc);
 
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
         CJS_EventHandler* pEvent = pContext->GetEventHandler();
         ASSERT(pEvent != NULL);
 
         if (params.size() != 1)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
         std::string cFormat;
-        int iIndex = (int)params[0];    
+        int iIndex = params[0].ToInt();
 
         if(!pEvent->m_pValue)
                 return FALSE;
         //CJS_Value val = pEvent->Value();
         CFX_WideString& val = pEvent->Value();
         std::string strSrc = CFX_ByteString::FromUnicode(val).c_str();
         std::wstring wstrChange(pEvent->Change());
         
         switch (iIndex) 
         {
@@ skipping 73 matching lines @@
 {
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
 
         if (params.size() != 2)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
-        CFX_WideString sValue = params[0].operator CFX_WideString();
-        CFX_WideString sFormat = params[1].operator CFX_WideString();
+        CFX_WideString sValue = params[0].ToCFXWideString();
+        CFX_WideString sFormat = params[1].ToCFXWideString();
 
         FX_BOOL bWrongFormat = FALSE;
         double dDate = MakeRegularDate(sValue,sFormat,bWrongFormat);
 
         if (JS_PortIsNan(dDate))
         {
                 CFX_WideString swMsg;
                 swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSPARSEDATE), sFormat.c_str());
                 Alert((CJS_Context *)cc, swMsg);
                 return FALSE;
         }
-        
+
         vRet = dDate;
-
         return TRUE;
 }
 
 FX_BOOL CJS_PublicMethods::AFSimple(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         if (params.size() != 3)
         {
                 CJS_Context* pContext = (CJS_Context *)cc;
                 ASSERT(pContext != NULL);
 
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
-        vRet = (double)AF_Simple(params[0].operator CFX_WideString(), (double)params[1], (double)params[2]);
+        vRet = (double)AF_Simple(params[0].ToCFXWideString(), params[1].ToDouble(), params[2].ToDouble());
         return TRUE;
 }
 
 FX_BOOL CJS_PublicMethods::AFMakeNumber(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         if (params.size() != 1)
         {
                 CJS_Context* pContext = (CJS_Context *)cc;
                 ASSERT(pContext != NULL);
 
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
-    vRet = ParseStringToNumber(params[0].operator CFX_WideString());
+        vRet = ParseStringToNumber(params[0].ToCFXWideString());
         return TRUE;
 }
 
 FX_BOOL CJS_PublicMethods::AFSimple_Calculate(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         v8::Isolate* isolate = ::GetIsolate(cc);
 
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
 
         if (params.size() != 2)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
         CJS_Value params1 = params[1];
 
         if (!params1.IsArrayObject() && params1.GetType() != VT_string)
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
-        
+
         CPDFSDK_Document* pReaderDoc = pContext->GetReaderDocument();
     ASSERT(pReaderDoc != NULL);
 
         CPDFSDK_InterForm* pReaderInterForm = pReaderDoc->GetInterForm();
         ASSERT(pReaderInterForm != NULL);
 
         CPDF_InterForm* pInterForm = pReaderInterForm->GetInterForm();
         ASSERT(pInterForm != NULL);
 
         double dValue;
-        CFX_WideString sFunction = params[0].operator CFX_WideString();
+        CFX_WideString sFunction = params[0].ToCFXWideString();
         if (wcscmp(sFunction, L"PRD") == 0)
         dValue = 1.0;
         else
                 dValue = 0.0;
 
         CJS_Array FieldNameArray = AF_MakeArrayFromList(isolate,params1);
 
         int nFieldsCount = 0;
 
         for (int i=0,isz=FieldNameArray.GetLength(); i<isz; i++)
         {
                 CJS_Value jsValue(isolate);
                 FieldNameArray.GetElement(i,jsValue);
-        CFX_WideString wsFieldName = jsValue.operator CFX_WideString();
+                CFX_WideString wsFieldName = jsValue.ToCFXWideString();
 
         for (int j=0,jsz=pInterForm->CountFields(wsFieldName); j<jsz; j++)
                 {
                         if (CPDF_FormField* pFormField = pInterForm->GetField(j, wsFieldName))
                         {
                                 double dTemp = 0.0;
 
                                 switch (pFormField->GetFieldType())
                                 {
                                 case FIELDTYPE_TEXTFIELD:
@@ skipping 50 matching lines @@
                         }
                 }
         }
 
         if (wcscmp(sFunction, L"AVG") == 0 && nFieldsCount > 0)
                 dValue /= nFieldsCount;
 
         dValue = (double)floor(dValue * FXSYS_pow((double)10,(double)6) + 0.49) / FXSYS_pow((double)10,(double)6);
         CJS_Value jsValue(isolate,dValue);
         if((CJS_EventHandler*)pContext->GetEventHandler()->m_pValue)
-                ((CJS_EventHandler*)pContext->GetEventHandler())->Value() = jsValue;
+                ((CJS_EventHandler*)pContext->GetEventHandler())->Value() = jsValue.ToCFXWideString();
 
         return TRUE;
 }
 
 /* This function validates the current event to ensure that its value is 
 ** within the specified range. */
 
 FX_BOOL CJS_PublicMethods::AFRange_Validate(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         CJS_Context* pContext = (CJS_Context *)cc;
         ASSERT(pContext != NULL);
         CJS_EventHandler* pEvent = pContext->GetEventHandler();
         ASSERT(pEvent != NULL);
 
         if (params.size() != 4) 
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
         if(!pEvent->m_pValue)
                 return FALSE;
         if (pEvent->Value().IsEmpty() )
                 return TRUE;
         double dEentValue = atof(CFX_ByteString::FromUnicode(pEvent->Value()));
-        FX_BOOL bGreaterThan, bLessThan;
-        double  dGreaterThan, dLessThan;
-    bGreaterThan = (FX_BOOL)params[0];
+        FX_BOOL bGreaterThan = params[0].ToBool();
+        double dGreaterThan = params[1].ToDouble();
+        FX_BOOL bLessThan = params[2].ToBool();
+        double dLessThan = params[3].ToDouble();
         CFX_WideString swMsg;
-        dGreaterThan = (double)params[1];
-        bLessThan = (FX_BOOL)params[2];
-        dLessThan = (double)params[3];
 
         if (bGreaterThan && bLessThan)
         {
                 if (dEentValue < dGreaterThan || dEentValue > dLessThan)
-                        swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSRANGE1),(FX_LPCWSTR)params[1].operator CFX_WideString(), (FX_LPCWSTR)params[3].operator CFX_WideString());
+                        swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSRANGE1),
+                                                 params[1].ToCFXWideString().c_str(),
+                                                 params[3].ToCFXWideString().c_str());
         }
         else if (bGreaterThan)
         {
                 if (dEentValue < dGreaterThan)
-                        swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSRANGE2), (FX_LPCWSTR)params[1].operator CFX_WideString());
+                        swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSRANGE2),
+                                                 params[1].ToCFXWideString().c_str());
         }
         else if (bLessThan)
         {
                 if (dEentValue > dLessThan)
-                        swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSRANGE3), (FX_LPCWSTR)params[3].operator CFX_WideString());
+                        swMsg.Format(JSGetStringFromID(pContext, IDS_STRING_JSRANGE3),
+                                                 params[3].ToCFXWideString().c_str());
         }
 
         if (!swMsg.IsEmpty())
         {
                 Alert(pContext, swMsg);
                 pEvent->Rc() = FALSE;
         }
         return TRUE;
 }
 
 FX_BOOL CJS_PublicMethods::AFExtractNums(IFXJS_Context* cc, const CJS_Parameters& params, CJS_Value& vRet, CFX_WideString& sError)
 {
         v8::Isolate* isolate = ::GetIsolate(cc);
         CJS_Context* pContext = (CJS_Context*)cc;
         ASSERT(pContext != NULL);
 
         if (params.size() != 1) 
         {
                 sError = JSGetStringFromID(pContext, IDS_STRING_JSPARAMERROR);
                 return FALSE;
         }
 
         CJS_Array nums(isolate);
 
-        CFX_WideString str = params[0].operator CFX_WideString();
+        CFX_WideString str = params[0].ToCFXWideString();
         CFX_WideString sPart;
 
         if (str.GetAt(0) == L'.' || str.GetAt(0) == L',')
                 str = L"0" + str;
 
         int nIndex = 0;
         for (int i=0, sz=str.GetLength(); i<sz; i++)
         {
                 FX_WCHAR wc = str.GetAt(i);
                 if (IsDigit((wchar_t)wc))
@@ skipping 16 matching lines @@
                 nums.SetElement(nIndex,CJS_Value(isolate,sPart.c_str()));       
         }
 
         if (nums.GetLength() > 0)
                 vRet = nums;
         else
                 vRet.SetNull();
 
         return TRUE;
 }