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

File/Image/Video field not associating files correctly #2955

Closed
brettweaver opened this issue May 20, 2015 · 7 comments
Closed

File/Image/Video field not associating files correctly #2955

brettweaver opened this issue May 20, 2015 · 7 comments
Assignees
Milestone

Comments

@brettweaver
Copy link

In an ACT Pod I have a field set to File/Image/Video type and it was working until today. I ran updates yesterday and now have v2.5.2. WordPress is at 4.2.2

Problem: When we select an image from the media library it should display a "pinkynail" image, but it doesn't. Saving a new record after selecting an image fails with a modal popup: "Error: Name is Empty". Adding an image to an existing record seems to work, although the pinkynail is not displayed until after saving. Before saving the record, the url of the image is "http://icon". Doeesn't seem right, but that may not be part of the issue. See screen shot attached.
2015-05-20_pods-image-error

@Shelob9 Shelob9 added Type: Bug Bug: Visual Status: Help Wanted We have not prioritized this yet, but you can help make it happen to speed it up labels May 21, 2015
@Shelob9 Shelob9 added this to the Pods 2.5.3 milestone May 21, 2015
@Shelob9
Copy link
Collaborator

Shelob9 commented May 21, 2015

@brettweaver Are you seeing any console errors when you set an image? A malformed AJAX response could be to blame here...

@brettweaver
Copy link
Author

Yes re: console error:

"Failed to load resource: net::ERR_NAME_NOT_RESOLVED http://icon/"

console-error

@Shelob9
Copy link
Collaborator

Shelob9 commented May 21, 2015

@brettweaver Yep, I see that too.

If you have a minute to test #2957 which should fix this, please let us know the results. That needs more testing before it is safe to merge.

@brettweaver
Copy link
Author

Sorry. If you're having trouble reproducing my "name empty" error it's because I was leaving the name empty. Go figure. I haven't actually USED this app in a while (client site), and I just forgot to add the name/title. Seems to be working from here.
Thanks Josh!

@pglewis
Copy link
Contributor

pglewis commented May 28, 2015

Is this verified fixed? I've submitted a 3.0 PR as well, so is this ready to be closed?

@Shelob9
Copy link
Collaborator

Shelob9 commented May 28, 2015

@pglewis -- In my testing it worked. My testing was not expansive, but @sc0ttkclark said that was enough and merged it to 2.x. So close this once its merged to 3.0

@pglewis
Copy link
Contributor

pglewis commented May 30, 2015

Merged to 3.0 so off she goes.

@pglewis pglewis closed this as completed May 30, 2015
@sc0ttkclark sc0ttkclark removed the Status: Help Wanted We have not prioritized this yet, but you can help make it happen to speed it up label Dec 29, 2015
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

4 participants