Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Either I'm missing something obvious, or I have a conflicting module somewhere. I see no place to specify which directory I want scanned, which I expect is the first thing we'd want to be able to do with this very promising module.
If I dpm the form in sourceForm() I see the directory form field, but it doesn't seem to appear.
I turned off Feeds Tamper, seeing that had caused problems for this module in the past.
Using 7.x-2.0-beta5+0-dev with Feeds 7.x-2.0-alpha8
Thanks!
Comment | File | Size | Author |
---|---|---|---|
#9 | Feeds_directory_fetcher-Fixed2_changing_file_mask_for_fetcher-2002244.patch | 439 bytes | Gregory Claeyssens |
#6 | Screen Shot 2014-02-20 at 15.16.58.png | 59.7 KB | Gregory Claeyssens |
Comments
Comment #1
darkodev CreditAttribution: darkodev commentedSorry about that late night miss. You specify a directory on import, not when configuring the importer.
Comment #2
marktheshark CreditAttribution: marktheshark commentedWill this work with periodic imports where the user will not be specifying a directory every time?
Comment #3
j2b CreditAttribution: j2b commentedI still can not change File mask and save new settings due to the same error - Please specify an upload directory.
I've attached importer to node, created node, and then tried to change .txt to .ics. Receiving the same error and can not change this. Are we missing something in configuration?
Another question, may be directory is not correct. Using Private file download method, indicating ../private folder outside Drupal DocRoot. In Fetcher settings, putting in "private://", as thinking this is a root of private file system folder, where are .ics files located. Web server has access to these folders without problem.
Can you please provide info, to overcome this problem?
Comment #4
j2b CreditAttribution: j2b commentedP.S. It turned out, that this node, to which fetcher is assigned, is not saved too. Kind of a magic circle happening:
1. creating form to specify directory (to let change File mask)
2. saving node (fetcher activated on node save)
3. getting an error, as there are no files found in indicated directory (due to fact, that should be looking for .ics files, but default configuration offer .txt files, which obviously are not there).
Comment #5
rudyard55 CreditAttribution: rudyard55 commentedI'm confused... this issue is "closed as designed"? I can't change the file mask to xml, it defaults to txt. So it that by design?
Comment #6
Gregory Claeyssens CreditAttribution: Gregory Claeyssens commentedSame problem here.
Comment #7
Gregory Claeyssens CreditAttribution: Gregory Claeyssens commentedSeems like the problem is because the class "feeds_fetcher_directory_fetcher" extends the class "FeedsFileFetcher" from the feeds plugin.
FeedsFileFetcher class hass following function wich should be overwritten:
Comment #8
Gregory Claeyssens CreditAttribution: Gregory Claeyssens commentedAdd this function in the class feeds_fetcher_directory_fetcher (located in file: "feeds_fetcher_directory/feeds_fetcher_directory_fetcher.inc"
And all seems good, i'll test this further and post a patch if it work correctly
Comment #9
Gregory Claeyssens CreditAttribution: Gregory Claeyssens commentedHere is the patch.
Comment #10
Frank Ralf CreditAttribution: Frank Ralf commentedApplied the patch and it solves the problem of not being able to change the file mask.
Many thanks, Gregory!
BTW
You can embed screen-shots directly in your posts, and when you use
tags
for your code snippets they will show with syntax-highlighting ;-)Comment #11
Anonymous (not verified) CreditAttribution: Anonymous commentedAnother RTBC for patch in #9, it worked nicely, please commit!
Comment #12
Niremizov CreditAttribution: Niremizov commentedComment #13
baaj CreditAttribution: baaj commentedConfirming that the patch in #9 works well, thanks!
Comment #14
Steven Jones CreditAttribution: Steven Jones at ComputerMinds commentedThanks for the patch!
Comment #16
Steven Jones CreditAttribution: Steven Jones at ComputerMinds commentedOh, pants, this conflicts with #2023775: configFormValidate() errors going to rollback in favour of that one!