Open brodycj opened 7 years ago
@brodybits there is any update on the timeline for this release? I'm interested particularly in the fix for the issue #563
Thanks
My apologies for the delay that was due to workload. I will probably need another 1-2 months and may combine with major release in #687.
@jabawack81 workaround for #563 is to fix it yourself in a fork or with help from npmjs.com / patch-package.
@brodybits I would love to if I knew what to fix.
@jabawack81 the /SAFESEH build flag for Win32 is now integrated with this plugin version as well as cordova-sqlite-ext. Please report if it helps or not.
cordova-sqlite-storage 2.1.0 & 2.1.1 released with some changes discussed here:
SQLITE_THREADSAFE=2
again ref: #703/SAFESH
as described in:
along with a couple additional changes from pull requests:
I Brody I got the same error.
My test procedure:
and got this error:
------ Building platform: windows
------ Build configuration options: --release -- --win --archs "x86 arm"
No scripts found for hook "before_compile".
PlatformApi successfully found for platform windows
Reading build config file: C:[PROJECTFOLDER]\build.json
Running command: "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\msbuild.exe" -version -nologo
Command finished with error code 0: C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\msbuild.exe -version,-nologo
Using MSBuild v15.1 from C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin
Building project: C:[PROJECTFOLDER]\platforms\windows\CordovaApp.Windows10.jsproj
Configuration : release
Platform : x86
Running command: C:\WINDOWS\system32\reg.exe query "HKLM\SOFTWARE\Microsoft\Microsoft SDKs\Windows" /s /v InstallationFolder /reg:32
Command finished with error code 0: C:\WINDOWS\system32\reg.exe query,HKLM\SOFTWARE\Microsoft\Microsoft SDKs\Windows,/s,/v,InstallationFolder,/reg:32
Running command: "C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\msbuild.exe" C:[PROJECTFOLDER]\platforms\windows\CordovaApp.Windows10.jsproj /clp:NoSummary;NoItemAndPropertyList;Verbosity=minimal /nologo /p:Configuration=release /p:Platform=x86 /p:AppxBundle=Never /p:UapAppxPackageBuildMode=StoreUpload
Patching 10 in prebuild event...
Injected base.js reference to the /www/index.html
Removing /( *)(<script\s+(?:type="text\/javascript"\s+)?src="\/\/Microsoft.WinJS.2.0\/js\/base.js">\s*<\/script>)(\s*)/ from /www/index.html
Removing /( *)(<script\s+(?:type="text\/javascript"\s+)?src="\/\/Microsoft.Phone.WinJS.2.1\/js\/base.js">\s*<\/script>)(\s*)/ from /www/index.html
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\Microsoft.Common.CurrentVersion.targets(4148,5): error MSB3030: Could not copy the file "C:[PROJECTFOLDER]\plugins\cordova-sqlite-storage\src\windows\SQLite3-Win-RT\SQLite3\SQLite3.UWP\Debug\SQLite3.UWP\SQLite3.winmd" because it was not found. [C:[PROJECTFOLDER]\platforms\windows\CordovaApp.Windows10.jsproj]
Command finished with error code 1: C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\msbuild.exe C:[PROJECTFOLDER]\platforms\windows\CordovaApp.Windows10.jsproj,/clp:NoSummary;NoItemAndPropertyList;Verbosity=minimal,/nologo,/p:Configuration=release,/p:Platform=x86,/p:AppxBundle=Never,/p:UapAppxPackageBuildMode=StoreUpload
1>MSBUILD : cordova-build error : Error: C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\msbuild.exe: Command failed with exit code 1
1>Done building project "project.jsproj" -- FAILED.
========== Build: 0 succeeded, 1 failed, 0 up-to-date, 0 skipped ==========
========== Package: 0 succeeded, 2 failed ===========
@jabawack81 I just updated https://github.com/brodybits/cordova-sqlite-storage-starter-app and was able to create a Windows app package from project -> store -> create app packages (with a test certificate, I did not create the app package for the app store but this part should not matter). In general cordova prepare
is not necessary if you do cordova platform add windows
with the plugins already configured in config.xml
. In case of any further issues:
Hi guys, any news about next release? I get bug on Windows 10 https://github.com/litehelpers/Cordova-sqlite-storage/pull/709 How can I help with next release that will include this fix?
Is there any plan or possibility that spatialite will be integrated or can be used with any of supported plugin versions?
Spatialite is already discussed in https://github.com/xpbrew/cordova-sqlite-storage-help/issues/32.
At this point I am giving top priority to a newly redesigned plugin version which is in progress at this time, discussed in https://github.com/brodybits/ask-me-anything/issues/3.
Priority for other items is given to paid customers. Please feel free to reach out to chris.brody+brodybits@gmail.com or sales@xpbrew.com in case of interest.
Minor release
Mandatory
ADDITIONAL (GENERAL):
Workaround solutions for iOS/macOS data loss risk issues:DONEcompile withDONE_SQLITE_THREADSAFE=2
again ref: #703workaround solution for iOS threading issue reported in #716DONEAddDONE/SAFESH
as described in:https://github.com/litehelpers/Cordova-sqlite-storage/issues/563#issuecomment-287648222https://msdn.microsoft.com/en-us/library/9a89h429.aspxDocumentation
ADDITIONAL:
Note that standard transaction mechanism is no longer recommended and to be deprecated ref: #690DONEDocument "multiple sqlite problem risk" ref: #626DONEFeature(s)
UPDATE: Moved to major release ref: #687 due to time/schedule constraint:
Function to get database directory path ref: litehelpers/Cordova-sqlite-storage#680 (may be affected by litehelpers/Cordova-sqlite-storage#681: Database storage in arbitrary path Android/iOS/macOS/Windows)Highly desired
Documentation
Testing
ADDITIONAL:
Fixes & features
UPDATE: these updates are expected to come in the major release ref: #687
ADDITIONAL, HIGHLY DESIRED before major release
now expected to come in the major release ref: #687[ ] Use UTF-8 encoding for Windows ref: litehelpers/Cordova-sqlite-storage#652
[ ]
TBD better cleanup of "new transaction is waiting for open operation" log messages ref: litehelpers/Cordova-sqlite-storage#679[ ] _SQLite3 compile options such as HAVEUSLEEP & NDEBUG for closer match to sqlcipher version
[ ] Integrate fix contributed in litehelpers/Cordova-sqlite-storage#709 to handle NULL (
u0000
) characters on Windows platform[ ] Should refuse to open new database files after
onDestroy
on Android has been called, equivalent on iOS ref: PR #697[ ] Update needed to work with cordova-android@7 ref: #729
[x]
builtin android.database end transaction if active before closing (8192bc841487252762cbeaf50587c1de30767186) ref:#730DONE[ ] Bind all numbers as REAL numbers ref: litehelpers/Cordova-sqlite-storage#684 possibly before major release
[ ] TBD (possibly related to litehelpers/Cordova-sqlite-storage#684): builtin android.database implementation seems to treat numeric parameter argument values as string values, needs investigation ref: https://github.com/litehelpers/Cordova-sqlite-storage/pull/730#issuecomment-350918898
[ ] _
TBD for investigation: PRAGMA journal_mode should be same value on builtin android.database implementation, default Android-sqlite-connector implementation, and other platforms ref: https://github.com/litehelpers/Cordova-sqlite-storage/pull/730#issuecomment-350919989WILL NOT BE CHANGED: PRAGMA journalmode reportsdelete
in case of default android.database implementation (same as other platform implementations) on Android 8.1[ ] Fix broken Circle/Travis CI or remove
[ ] JavaScript remove qid no longer needed & other cleanup
[ ] Fix for builtin android.database implementation on Turkish and other foreign locales in PR #713
[x] PR #657: Add plugin typings from DefinitelyTyped
[ ] address iOS/macOS resource leak when deleting database without closing first
Use extra durable build option as described in https://github.com/litehelpers/Cordova-sqlite-storage/issues/736#issuecomment-357123554
review any other possible causes of database corruption
ref: #736TBDiOS/macOS platform version openDatabase checks from https://github.com/brodybits/cordova-sqlite-evmax-legacy-exp-free
Resolve issue with PSPDFKit ref: https://github.com/litehelpers/Cordova-sqlcipher-adapter/issues/59
Windows implementation should include an error object when signaling a close error
Resolve INCORRECT error code / INCONSISTENT error messages on Windows
TBD include actual error code, perhaps in error message (already done on builtin android.database implementation in case of androidDatabaseImplementation: 2 on Android)
Include info about prepare statement error vs execute statement error, as done on Windows
_testing with PRAGMA foreignkeys ref: https://github.com/litehelpers/Cordova-sqlite-evcore-extbuild-free/issues/23
test/document issue with Unicode column name ref: #641.
improve testing & documentation related to Unicode characters on Android ref: #739
document existence of spatialite fork
pointer to https://github.com/Crypho/cordova-plugin-secure-storage
ADDITIONAL DOCUMENTATION:
MAJOR DOCUMENTATION CLEANUP NEEDED here or in NEXT MAJOR RELEASE (#687) ref: https://github.com/litehelpers/Cordova-sqlite-help/issues/33 & https://github.com/litehelpers/Cordova-sqlite-storage/issues/596#issuecomment-351980903
Timeline
4-6 weeks due to existing backlog