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

[studio] File selector with upload data source does not report file size #608

Closed
russdanner opened this issue Mar 16, 2017 · 3 comments
Closed

Comments

@russdanner
Copy link
Member

russdanner commented Mar 16, 2017

The file selector component no longer receives the file size from the backend when uploading a file:

Data Source: file-desktop-upload.js

Calls this operation:
CStudioAuthoring.Operations.uploadAsset(site, path, isUploadOverwrite, callback);

Which on success returns fileData
success: function(fileData) { if (control) { control.insertItem(path + "/" + fileData.fileName, path + "/" + fileData.fileName, fileData.fileExtension, fileData.size); control._renderItems(); } }

fileData.size is returning undefined.

To reproduce:

Create a simple form with a file-upload data source linked to a item selector, open form and upload file.

Note that call to service does NOT return size in response. It should.
http://localhost:8080/studio/asset-upload?nocache=Fri%20Mar%2024%202017%2011:05:44%20GMT-0400%20(EDT)
{ fileName: "research.jpg", fileExtension: "jpg" }

@sumerjabri
Copy link
Member

@russdanner can you provide details on how to reach this state and more clearly indicate what's wrong?

@sumerjabri
Copy link
Member

Closing this as we don't have enough detail.

@russdanner russdanner reopened this Mar 24, 2017
@sumerjabri sumerjabri assigned dejan-brkic and unassigned sumerjabri Mar 24, 2017
@sumerjabri sumerjabri added the bug label Mar 24, 2017
@sumerjabri sumerjabri modified the milestones: Crafter CMS v3.0, Crafter CMS v2.5.6 Mar 24, 2017
@sumerjabri sumerjabri added this to In Progress in Crafter CMS v3.0 Mar 24, 2017
@dejan-brkic
Copy link
Member

Done
studio:
PR #601 #602

studio-ui
PR #308 #309

@dejan-brkic dejan-brkic moved this from In Progress to Test & Validate in Crafter CMS v3.0 Mar 24, 2017
@dejan-brkic dejan-brkic added this to Testing in Crafter CMS v2.5.x Mar 24, 2017
@sumerjabri sumerjabri moved this from Test & Validate to Completed in Crafter CMS v3.0 Mar 28, 2017
@russdanner russdanner moved this from Testing to Completed in Crafter CMS v2.5.x Sep 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Crafter CMS v2.5.x
  
Completed
Crafter CMS v3.0
  
Completed
Development

No branches or pull requests

3 participants