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.

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

Make this complete to run tests locally. #638

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

username1565
Copy link

Add latest smallest jquery-3.4.1.slim.min.js
Fix pathways in ./index.html. Now, ./index.html working after run this from folder.
Add /dist/jszip-utils.js
fix protocols and pathways in ./test/index.html. Add warning there, in the top of the page.

Add latest smallest jquery-3.4.1.slim.min.js
Fix pathways in ./index.html. Now, ./index.html working after run this from folder.
Add /dist/jszip-utils.js
fix protocols and pathways in ./test/index.html. Add warning there, in the top of the page.
@username1565
Copy link
Author

Maybe, need to add another scripts from ./test/index.html, which are loading from another resources. After many years, this resources can be not available, but if this scripts will be downloaded and added to repositary, then jszip will be complete.

Because Jquery already exists in `./test/jquery-1.8.3.min.js`
Replace old Jquery `./test/vendor/jquery-1.8.3.min.js` (93,637 bytes)
to latest minified of JQuery-slim: `./test/vendor/jquery-3.4.1.slim.min.js` (71,037 bytes).

Add `qunit-1.23.0.js` and `qunit-1.23.0.css` to the `./test/vendor`-folder.

Change pathways for this files, in `index.html` and `./test/index.html`.
@username1565 username1565 changed the title Make this complete to run teste locally. Make this complete to run tests locally. Nov 23, 2019
@username1565
Copy link
Author

Done. Now this code may to be complete.

Changes:
	Added windows-1251 and DOS(cp866) encoding to correct encode-decode the ciryllic symbols on Windows.
		Now this specified in params, as
			{encoding: "windows-1251"},
			{encoding: "Windows-1251"},
			{encoding: "cp866"},
	Added third case:
		{encoding: "pathways_and_filenames-encoding-is-cp866_password-encoding-is-windows-1251"},
		when pathway encoding is cp866 but encoding for password is windows-1251.

	Default encoding for txt-file content is not changed, but now this can be a string, not buffer of bytearray.
	
	Maybe all encoders need to be added in separate module and .js-script, and using only hex(bytes), as values on input-ouptut.
	
	Here, you can see cyrillic-symbols in pathway (filename), and password.
	This is compressing, and uncompressing sucessfully, by using this script.
	
	abc_пароль.zip - zip-file, which was been generated by this script.
	
	make_array_with_bytes.html - the page to return hexdump - "array_with_bytes.txt", after uploding binary file.
	
	smaller_array_with_bytes.txt - array, without many "0x00" bytes, in the end.


	
	I don't know how to write this in C-code, because as I understand, minizip-asm.min.js is generated from C-code.
	
	But anyway, anyone can read this, understand this, see differences, and fix this by more qualified way.
@username1565
Copy link
Author

Sorry, previous files was been uploaded here, by my mistake.
I fixed encoding in another project: https://github.com/username1565/minizip-asm.js/
Here is this commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant