apply review commnets
This commit is contained in:
parent
bedfd462f7
commit
12f5e727b5
|
@ -169,7 +169,7 @@ class ColumnMeta
|
||||||
public function isEncryptableType()
|
public function isEncryptableType()
|
||||||
{
|
{
|
||||||
$unsupportedTypes = array("money", "smallmoney", "image", "ntext", "text", "xml", "sql_variant");
|
$unsupportedTypes = array("money", "smallmoney", "image", "ntext", "text", "xml", "sql_variant");
|
||||||
if (in_array(strtolower($this->dataType), $unsupportedTypes) && !$this->forceEncrypt) {
|
if (!$this->forceEncrypt && in_array(strtolower($this->dataType), $unsupportedTypes)) {
|
||||||
return false;
|
return false;
|
||||||
} else {
|
} else {
|
||||||
return true;
|
return true;
|
||||||
|
|
|
@ -10,6 +10,7 @@ require_once("MsCommon_mid-refactor.inc");
|
||||||
require_once("AEData.inc");
|
require_once("AEData.inc");
|
||||||
$dataTypes = array("smallmoney", "money");
|
$dataTypes = array("smallmoney", "money");
|
||||||
try {
|
try {
|
||||||
|
//set to ERRMODE_SILENT to compare actual error and expected unsupport money types in encrypted columns error
|
||||||
$conn = connect('', array(), PDO::ERRMODE_SILENT);
|
$conn = connect('', array(), PDO::ERRMODE_SILENT);
|
||||||
foreach ($dataTypes as $dataType) {
|
foreach ($dataTypes as $dataType) {
|
||||||
echo "\nTesting $dataType:\n";
|
echo "\nTesting $dataType:\n";
|
||||||
|
|
|
@ -10,6 +10,7 @@ require_once("MsCommon_mid-refactor.inc");
|
||||||
require_once("AEData.inc");
|
require_once("AEData.inc");
|
||||||
$dataTypes = array( "smallmoney", "money" );
|
$dataTypes = array( "smallmoney", "money" );
|
||||||
try {
|
try {
|
||||||
|
//set to ERRMODE_SILENT to compare actual error and expected unsupport money types in encrypted columns error
|
||||||
$conn = connect('', array(), PDO::ERRMODE_SILENT);
|
$conn = connect('', array(), PDO::ERRMODE_SILENT);
|
||||||
foreach ($dataTypes as $dataType) {
|
foreach ($dataTypes as $dataType) {
|
||||||
echo "\nTesting $dataType:\n";
|
echo "\nTesting $dataType:\n";
|
||||||
|
@ -60,7 +61,7 @@ try {
|
||||||
if ($success) {
|
if ($success) {
|
||||||
echo "Test successfully done.\n";
|
echo "Test successfully done.\n";
|
||||||
}
|
}
|
||||||
DropTable($conn, $tbname);
|
dropTable($conn, $tbname);
|
||||||
}
|
}
|
||||||
unset($stmt);
|
unset($stmt);
|
||||||
unset($conn);
|
unset($conn);
|
||||||
|
|
|
@ -11,11 +11,11 @@ try {
|
||||||
$conn = connect();
|
$conn = connect();
|
||||||
// Create the table
|
// Create the table
|
||||||
$tbname = 'Patients';
|
$tbname = 'Patients';
|
||||||
$colMetaArr = array( new columnMeta("int", "PatientId", "IDENTITY(1,1)"),
|
$colMetaArr = array( new ColumnMeta("int", "PatientId", "IDENTITY(1,1)"),
|
||||||
new columnMeta("char(11)", "SSN"),
|
new ColumnMeta("char(11)", "SSN"),
|
||||||
new columnMeta("nvarchar(50)", "FirstName", "NULL"),
|
new ColumnMeta("nvarchar(50)", "FirstName", "NULL"),
|
||||||
new columnMeta("nvarchar(50)", "LastName", "NULL"),
|
new ColumnMeta("nvarchar(50)", "LastName", "NULL"),
|
||||||
new columnMeta("date", "BirthDate", null, "randomized"));
|
new ColumnMeta("date", "BirthDate", null, "randomized"));
|
||||||
createTable($conn, $tbname, $colMetaArr);
|
createTable($conn, $tbname, $colMetaArr);
|
||||||
|
|
||||||
// insert a row
|
// insert a row
|
||||||
|
|
|
@ -38,7 +38,7 @@ if (isColEncrypted()) {
|
||||||
$conn1 = connect('', array(), PDO::ERRMODE_EXCEPTION, true);
|
$conn1 = connect('', array(), PDO::ERRMODE_EXCEPTION, true);
|
||||||
$stmt = $conn1->query($selectSql);
|
$stmt = $conn1->query($selectSql);
|
||||||
while ($decrypted_row = $stmt->fetch(PDO::FETCH_ASSOC)) {
|
while ($decrypted_row = $stmt->fetch(PDO::FETCH_ASSOC)) {
|
||||||
if ($decrypted_row[ 'CharCount' ] == strlen($decrypted_row[getDefaultColName("nvarchar(1000)")])) {
|
if ($decrypted_row['CharCount'] == strlen($decrypted_row[getDefaultColName("nvarchar(1000)")])) {
|
||||||
$rowInd = $decrypted_row[ 'CharCount' ] + 1;
|
$rowInd = $decrypted_row[ 'CharCount' ] + 1;
|
||||||
echo "Failed to encrypted at row $rowInd\n";
|
echo "Failed to encrypted at row $rowInd\n";
|
||||||
$testPass = false;
|
$testPass = false;
|
||||||
|
|
|
@ -16,7 +16,7 @@ try {
|
||||||
|
|
||||||
// create table
|
// create table
|
||||||
$tbname = getTableName();
|
$tbname = getTableName();
|
||||||
$colMetaArr = array( new columnMeta($dataType, "c_det"), new columnMeta($dataType, "c_rand", null, "randomized"));
|
$colMetaArr = array(new ColumnMeta($dataType, "c_det"), new ColumnMeta($dataType, "c_rand", null, "randomized"));
|
||||||
createTable($conn, $tbname, $colMetaArr);
|
createTable($conn, $tbname, $colMetaArr);
|
||||||
|
|
||||||
// insert a row
|
// insert a row
|
||||||
|
|
Loading…
Reference in a new issue