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

Broken file when trying to copy a file to folder. #17241

Closed
fabrizzio-dotCMS opened this issue Sep 12, 2019 · 6 comments
Closed

Broken file when trying to copy a file to folder. #17241

fabrizzio-dotCMS opened this issue Sep 12, 2019 · 6 comments

Comments

@fabrizzio-dotCMS
Copy link
Contributor

On site-browser Create a file asset upload an image. Hit save and publish.
Then locate the file recently created. use the contextual menu option Mark-for-copy

Select another folder then again from the contextual menu chose paste

On this step, the exception isn't evident and it looks like the file was copied successfully.

if you attempt again to perform another copy by marking the file (the one we just pasted) and use again the contextual menu option Mark-for-copy then select another folder and hit paste again you'll see an exception like :

ERROR ajax.BrowserAjax - Error trying to copy the file to folder.
com.dotmarketing.portlets.contentlet.business.DotContentletStateException: Cannot checkin 0 length file: /Users/fabrizzio/code/servers/server1/assets/tmp_upload/dotcms.org.1/actn-legioss.jpg

The file exists on that path but it's empty.

@wezell wezell added this to the Cody Current milestone Sep 12, 2019
@wezell
Copy link
Contributor

wezell commented Sep 12, 2019

Need to fix this. What code.path is causing this?

@fabrizzio-dotCMS
Copy link
Contributor Author

zacatapum

@fabrizzio-dotCMS
Copy link
Contributor Author

16:46:21.393 ERROR ajax.BrowserAjax - Error trying to copy the file to folder. com.dotmarketing.portlets.contentlet.business.DotContentletStateException: Cannot checkin 0 length file: /Users/fabrizzio/code/servers/server1/assets/tmp_upload/dotcms.org.1/coal-miners-family.jpg at com.dotcms.content.elasticsearch.business.ESContentletAPIImpl.internalCheckin(ESContentletAPIImpl.java:3888) ~[dotcms_5.0.0_999999.jar:?] at com.dotcms.content.elasticsearch.business.ESContentletAPIImpl.lambda$28(ESContentletAPIImpl.java:3296) ~[dotcms_5.0.0_999999.jar:?] at com.dotcms.concurrent.lock.StripedLockImpl.tryLock(StripedLockImpl.java:100) ~[dotcms_5.0.0_999999.jar:?] at com.dotcms.concurrent.lock.StripedLockImpl.tryLock(StripedLockImpl.java:55) ~[dotcms_5.0.0_999999.jar:?] at com.dotcms.concurrent.lock.IdentifierStripedLock.tryLock(IdentifierStripedLock.java:16) ~[dotcms_5.0.0_999999.jar:?] at com.dotcms.content.elasticsearch.business.ESContentletAPIImpl.checkin_aroundBody138(ESContentletAPIImpl.java:3295) ~[dotcms_5.0.0_999999.jar:?] at com.dotcms.content.elasticsearch.business.ESContentletAPIImpl$AjcClosure139.run(ESContentletAPIImpl.java:1) ~[dotcms_5.0.0_999999.jar:?] at org.aspectj.runtime.reflect.JoinPointImpl.proceed(JoinPointImpl.java:149) ~[aspectjrt-1.8.10.jar:?]

jgambarios added a commit that referenced this issue Sep 17, 2019
jgambarios added a commit that referenced this issue Sep 17, 2019
@jgambarios
Copy link
Contributor

PR: #17272

@jdotcms
Copy link
Contributor

jdotcms commented Sep 25, 2019

The case is fixed, it is just taking some time to be reflected on the site browser probably b/c it is using DEFER

@bryanboza
Copy link
Member

Fixed, tested on master // Postgres // FF

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

No branches or pull requests

5 participants