Asterix is missing for 'file' field which is a required field.

Comments

shwetaneelsharma created an issue. See original summary.

alansaviolobo’s picture

StatusFileSize
new38.64 KB

The entire feed section is generated by the feeds module and hence this issue would have to be taken up in the feeds issue queue.
feeds section

alansaviolobo’s picture

Project: Giveaways » Feeds
Version: 7.x-1.x-dev » 7.x-2.x-dev
Component: Documentation » Code
Assigned: alansaviolobo » Unassigned
megachriz’s picture

In your screenshot, a file is already uploaded. So uploading an other file is not required. I think this works as designed.

alansaviolobo’s picture

StatusFileSize
new57.98 KB

Apologies for the confusion. Here is the correct screenshot
create node form

megachriz’s picture

Status: Active » Needs review
StatusFileSize
new496 bytes

Thanks for the other screenshot!

This patch could fix it. Lets see if this breaks any tests.

Status: Needs review » Needs work

The last submitted patch, 6: feeds7-file-upload-required-2832092-6.patch, failed testing.

megachriz’s picture

Apparently, it does break tests. Either the fix isn't completely right, or there is are good reason that the file upload field is not marked as required.

megachriz’s picture

A field of type 'file' cannot be set as required due to this Drupal core bug: #59750: Required flag on file forms breaks on validation.

Converting the file upload field to the field type 'managed_file' could fix the problem as said in #59750-51: Required flag on file forms breaks on validation. This may require more changes in the file fetcher.

bluegeek9’s picture

Status: Needs work » Closed (outdated)

Drupal 7 reached end of life and the D7 version of Feeds is no longer being developed. To keep the issue queue focused on supported versions, we’re closing older D7 issues.

If you still have questions about using Feeds on Drupal 7, feel free to ask. While we won’t fix D7 bugs anymore, we’re happy to offer guidance to help you move forward. You can do so by opening (or reopening) a D7 issue, or by reaching out in the #feeds channel on Drupal Slack.

If this issue is still relevant for Drupal 10+, please open a follow-up issue or merge request with proposed changes. Contributions are always welcome!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.