Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dev #1296

Merged
merged 29 commits into from
Sep 8, 2021
Merged

Dev #1296

merged 29 commits into from
Sep 8, 2021

Conversation

yitam
Copy link
Contributor

@yitam yitam commented Sep 8, 2021

No description provided.

yitam and others added 29 commits February 4, 2021 20:03
@yitam
Moved metadata cleanup to parent class (#1233)
3da8418
@yitam
Fixed two column encryption tests to encrypt columns (#1236)
f7e24bd
@morozov
Use lower-case object names in PDO::lastInsertId($name) (#1245)
fcd7b64
@yitam
Simplified implementations of last insert id and quote (#1251)
7fad440
@yitam
Simplified pdo quote and lastinsertid implementation further (#1254)
e57d5aa
@yitam
Updated code coverage excludes and ODBC driver (#1256)
a06e30d
@yitam
Refactored parameter processing and handling (#1239)
3bc0624
@yitam
Updated pdo_sqlsrv_get_driver_methods as per documentation (#1259)
7313fa0
@yitam
Cleaned up redundant code (#1260)
5f2a14c
@yitam
Implemented table-valued parameter support (#1257)
0da75f5
@yitam
Simplified parse_param_array in sqlsrv (#1262)
b49cb51
@yitam
Changed how schema is provided for TVP input (#1264)
a14cb70
@yitam
Simplified get_field_as_string and made it more robust (#1265)
af61d06
@yitam
Replaced ZVAL_NEW_ARR with array_init (#1267)
c91fe5c
@yitam
Fixed build script and failing tests (#1268)
d255591
@yitam
Updated BVT tests to test against only one database (#1273)
66b7ced
@yitam
Updated Readme file with useful links (#1274)
cc92fc5
@yitam @mlocati
Fix warning compiling core_stmt.cpp (#1275) (#1277)
b3b2ace 
converting to non-pointer type 'long int' from NULL

Co-authored-by: Michele Locati <michele@locati.it>
@yitam
Modified existing PDO tests for PHP 8.0+ (#1279)
9f6ef12
@yitam
Replaced vector with map for TVP constituent columns (#1281)
2778726
@yitam
PHP 8.1 dev (#1282)
5e607a8
@yitam
Made some tests more robust and work with PHP 8.1 (#1285)
fd48bcb
@yitam
Replaced strftime with IntlDateFormatter (#1286)
656767a
@yitam
Fixed the helper method for inserting nulls to binary fields (#1287)
9eef0b9
@yitam
Applied mask to pdo quote for binary inputs (#1288)
fffd63f
@yitam
Updated list of supported processor architecture (#1290)
f66b2c3
@yitam
Fixed snprintf for binary encoding and EOLs (#1293)
610f54c
@yitam
Update change log and version for 5.10.0 beta1 (#1294)
142ca11
@yitam
Updated Linux and mac instructions (#1295)
4af55d5
@codecov
Copy link

codecov bot commented Sep 8, 2021 •
edited
Loading

Codecov Report

Merging #1296 (4af55d5) into master (6d2b938) will increase coverage by 0.56%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1296      +/-   ##
==========================================
+ Coverage   82.81%   83.38%   +0.56%     
==========================================
  Files          24       22       -2     
  Lines        7222     7792     +570     
==========================================
+ Hits         5981     6497     +516     
- Misses       1241     1295      +54
Impacted Files Coverage Δ
...16/x86/php-8.0.0RC2-src/ext/pdo_sqlsrv/pdo_dbh.cpp
...6/x86/php-8.0.0RC2-src/ext/pdo_sqlsrv/pdo_stmt.cpp
...6/x86/php-8.0.0RC2-src/ext/sqlsrv/php_sqlsrv_int.h
...pdev/vs16/x86/php-8.0.0RC2-src/ext/sqlsrv/init.cpp
...x86/php-8.0.0RC2-src/ext/pdo_sqlsrv/pdo_parser.cpp
...p-8.0.0RC2-src/ext/pdo_sqlsrv/php_pdo_sqlsrv_int.h
....0.0RC2-src/ext/pdo_sqlsrv/shared/core_results.cpp
...6/php-8.0.0RC2-src/ext/sqlsrv/shared/core_util.cpp
...pdev/vs16/x86/php-8.0.0RC2-src/ext/sqlsrv/conn.cpp
...6/x86/php-8.0.0RC2-src/ext/pdo_sqlsrv/pdo_util.cpp
... and 36 more

@yitam yitam merged commit f2c3408 into master Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yitam @morozov