Hi

Add this line before Set filesObj = dirObj.Files
Set dirObj = mergeObj.Getfolder(dirObj.self.Path)

Cheers!!


+++++
*I did not do this for you. God is here working through me for you.*

On Tue, Dec 23, 2014 at 12:40 AM, Zafar Iqbal <ziqba...@gmail.com> wrote:

> Dear Vaibhav Joshi, I checked it and found it working till selection of
> Folder after that it is stuck at line Set filesObj = dirObj.Files .Please
> help to fix this error mentioned below. Thanks
>
> Run-time error '438':
> Object does not support this property or method
>
>
> Regards,
> ZAFAR IQBAL
> ---------------------------------------------------------------
>
> On Mon, Dec 22, 2014 at 5:20 PM, Vaibhav Joshi <v...@vabs.in> wrote:
>
>> Ok
>>
>> Try this...
>>
>>
>> Sub Data_Merge_From_All_Files()
>> Dim bookList As Workbook
>> Dim mergeObj, dirObj, filesObj, everyObj As Object
>> Application.ScreenUpdating = False
>> Set mergeObj = CreateObject("Scripting.FileSystemObject")
>>
>> 'Set dirObj = mergeObj.Getfolder("D:\Test")
>>
>> Set objShell = CreateObject("Shell.Application")
>> Set dirObj = objShell.BrowseForFolder(0, "Select Folder", 0,
>> myStartFolder)
>>
>> Set filesObj = dirObj.Files
>>
>> For Each everyObj In filesObj
>> If Right$(everyObj, 3) = "xls" Or Left$(Right$(everyObj, 4), 3) = "xls"
>> Then
>> Set bookList = Workbooks.Open(everyObj)
>> Range("A2:Q50" & Range("A65536").End(xlUp).Row).Copy
>> ThisWorkbook.Worksheets(1).Activate
>> Range("A1").Select
>> Selection.PasteSpecial Paste:=xlPasteColumnWidths, Operation:=xlNone, _
>>         SkipBlanks:=False, Transpose:=False
>>
>> Range("A65536").End(xlUp).Offset(1, 0).PasteSpecial
>> Application.CutCopyMode = False
>> bookList.Close
>> End If
>> Next
>> End Sub
>>
>>
>> Cheers!
>>
>> +++++
>> *I did not do this for you. God is here working through me for you.*
>>
>> On Mon, Dec 22, 2014 at 7:16 PM, Zafar Iqbal <ziqba...@gmail.com> wrote:
>>
>>> Dear Vaibhav Joshi,
>>>
>>> I think my query was not well written so needs to repeat it. This code
>>> is working fine for all files at fixed folder location written in code
>>> as "D:\Test". I want to make it flexible. Instead of manually writing
>>> folder location, File Open Dialog window should pop up enabling choose any
>>> folder within driver & select single or multiple excel file for processing
>>> by code. I think that if Application.GetOpenFilename(filefilter:="Excel
>>> Files (*.xl*), *.xl*",MultiSelect:=True) is used properly it will give
>>> this flexibilty. Please help. thanks
>>>
>>> Regards,
>>> ZAFAR IQBAL
>>> ---------------------------------------------------------------
>>>
>>> On Mon, Dec 22, 2014 at 4:23 PM, Vaibhav Joshi <v...@vabs.in> wrote:
>>>
>>>> Hi Zafar,
>>>>
>>>> Try this:
>>>>
>>>>
>>>> Sub Data_Merge_From_All_Files()
>>>> Dim bookList As Workbook
>>>> Dim mergeObj, dirObj, filesObj, everyObj As Object
>>>> Application.ScreenUpdating = False
>>>> Set mergeObj = CreateObject("Scripting.FileSystemObject")
>>>>
>>>> Set dirObj = mergeObj.Getfolder("D:\Test")
>>>> Set filesObj = dirObj.Files
>>>>
>>>> For Each everyObj In filesObj
>>>> If Right$(everyObj, 3) = "xls" Or Left$(Right$(everyObj, 4), 3) = "xls"
>>>> Then
>>>> Set bookList = Workbooks.Open(everyObj)
>>>> Range("A2:Q50" & Range("A65536").End(xlUp).Row).Copy
>>>> ThisWorkbook.Worksheets(1).Activate
>>>> Range("A1").Select
>>>> Selection.PasteSpecial Paste:=xlPasteColumnWidths, Operation:=xlNone, _
>>>>         SkipBlanks:=False, Transpose:=False
>>>>
>>>> Range("A65536").End(xlUp).Offset(1, 0).PasteSpecial
>>>> Application.CutCopyMode = False
>>>> bookList.Close
>>>> End If
>>>> Next
>>>> End Sub
>>>>
>>>>
>>>> Cheers!!
>>>>
>>>> +++++
>>>> *I did not do this for you. God is here working through me for you.*
>>>>
>>>> On Mon, Dec 22, 2014 at 12:32 PM, Zafar Iqbal <ziqba...@gmail.com>
>>>> wrote:
>>>>
>>>>> Dear Experts,
>>>>> Below mentioned code is to copy data from Sheet1 of all files at
>>>>> specified location and paste in active Sheet. For regular up-dation on
>>>>> weekly basis, it is needed to make this macro flexible with use of ADO
>>>>> Connection. It will allow us to choose required folder, select one or more
>>>>> latest files with shift/control button and get data pasted below last row
>>>>> of used range of current sheet. I tried to use below ADO option but could
>>>>> not get success. Please help to use it in below mentioned code. Thanks
>>>>>
>>>>> Application.GetOpenFilename(filefilter:="Excel Files (*.xl*),
>>>>> *.xl*",MultiSelect:=True)
>>>>>
>>>>> -----------------------------
>>>>>
>>>>> Sub Data_Merge_From_All_Files()
>>>>> Dim bookList As Workbook
>>>>> Dim mergeObj, dirObj, filesObj, everyObj As Object
>>>>> Application.ScreenUpdating = False
>>>>> Set mergeObj = CreateObject("Scripting.FileSystemObject")
>>>>>
>>>>> Set dirObj = mergeObj.Getfolder("D:\Test")
>>>>> Set filesObj = dirObj.Files
>>>>> For Each everyObj In filesObj
>>>>> Set bookList = Workbooks.Open(everyObj)
>>>>>
>>>>> Range("A2:Q50" & Range("A65536").End(xlUp).Row).Copy
>>>>> ThisWorkbook.Worksheets(1).Activate
>>>>> Range("A1").Select
>>>>> Selection.PasteSpecial Paste:=xlPasteColumnWidths, Operation:=xlNone, _
>>>>>         SkipBlanks:=False, Transpose:=False
>>>>>
>>>>> Range("A65536").End(xlUp).Offset(1, 0).PasteSpecial
>>>>> Application.CutCopyMode = False
>>>>> bookList.Close
>>>>> Next
>>>>> End Sub
>>>>>
>>>>> --
>>>>> Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be?
>>>>> It’s =TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @
>>>>> https://www.facebook.com/discussexcel
>>>>>
>>>>> FORUM RULES
>>>>>
>>>>> 1) Use concise, accurate thread titles. Poor thread titles, like
>>>>> Please Help, Urgent, Need Help, Formula Problem, Code Problem, and Need
>>>>> Advice will not get quick attention or may not be answered.
>>>>> 2) Don't post a question in the thread of another member.
>>>>> 3) Don't post questions regarding breaking or bypassing any security
>>>>> measure.
>>>>> 4) Acknowledge the responses you receive, good or bad.
>>>>> 5) Jobs posting is not allowed.
>>>>> 6) Sharing copyrighted material and their links is not allowed.
>>>>>
>>>>> NOTE : Don't ever post confidential data in a workbook. Forum owners
>>>>> and members are not responsible for any loss.
>>>>> ---
>>>>> You received this message because you are subscribed to the Google
>>>>> Groups "MS EXCEL AND VBA MACROS" group.
>>>>> To unsubscribe from this group and stop receiving emails from it, send
>>>>> an email to excel-macros+unsubscr...@googlegroups.com.
>>>>> To post to this group, send email to excel-macros@googlegroups.com.
>>>>> Visit this group at http://groups.google.com/group/excel-macros.
>>>>> For more options, visit https://groups.google.com/d/optout.
>>>>>
>>>>
>>>>  --
>>>> Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be?
>>>> It’s =TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @
>>>> https://www.facebook.com/discussexcel
>>>>
>>>> FORUM RULES
>>>>
>>>> 1) Use concise, accurate thread titles. Poor thread titles, like Please
>>>> Help, Urgent, Need Help, Formula Problem, Code Problem, and Need Advice
>>>> will not get quick attention or may not be answered.
>>>> 2) Don't post a question in the thread of another member.
>>>> 3) Don't post questions regarding breaking or bypassing any security
>>>> measure.
>>>> 4) Acknowledge the responses you receive, good or bad.
>>>> 5) Jobs posting is not allowed.
>>>> 6) Sharing copyrighted material and their links is not allowed.
>>>>
>>>> NOTE : Don't ever post confidential data in a workbook. Forum owners
>>>> and members are not responsible for any loss.
>>>> ---
>>>> You received this message because you are subscribed to a topic in the
>>>> Google Groups "MS EXCEL AND VBA MACROS" group.
>>>> To unsubscribe from this topic, visit
>>>> https://groups.google.com/d/topic/excel-macros/DrTphcwoTcA/unsubscribe.
>>>> To unsubscribe from this group and all its topics, send an email to
>>>> excel-macros+unsubscr...@googlegroups.com.
>>>> To post to this group, send email to excel-macros@googlegroups.com.
>>>> Visit this group at http://groups.google.com/group/excel-macros.
>>>> For more options, visit https://groups.google.com/d/optout.
>>>>
>>>
>>>  --
>>> Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be?
>>> It’s =TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @
>>> https://www.facebook.com/discussexcel
>>>
>>> FORUM RULES
>>>
>>> 1) Use concise, accurate thread titles. Poor thread titles, like Please
>>> Help, Urgent, Need Help, Formula Problem, Code Problem, and Need Advice
>>> will not get quick attention or may not be answered.
>>> 2) Don't post a question in the thread of another member.
>>> 3) Don't post questions regarding breaking or bypassing any security
>>> measure.
>>> 4) Acknowledge the responses you receive, good or bad.
>>> 5) Jobs posting is not allowed.
>>> 6) Sharing copyrighted material and their links is not allowed.
>>>
>>> NOTE : Don't ever post confidential data in a workbook. Forum owners and
>>> members are not responsible for any loss.
>>> ---
>>> You received this message because you are subscribed to the Google
>>> Groups "MS EXCEL AND VBA MACROS" group.
>>> To unsubscribe from this group and stop receiving emails from it, send
>>> an email to excel-macros+unsubscr...@googlegroups.com.
>>> To post to this group, send email to excel-macros@googlegroups.com.
>>> Visit this group at http://groups.google.com/group/excel-macros.
>>> For more options, visit https://groups.google.com/d/optout.
>>>
>>
>>  --
>> Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be? It’s
>> =TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @
>> https://www.facebook.com/discussexcel
>>
>> FORUM RULES
>>
>> 1) Use concise, accurate thread titles. Poor thread titles, like Please
>> Help, Urgent, Need Help, Formula Problem, Code Problem, and Need Advice
>> will not get quick attention or may not be answered.
>> 2) Don't post a question in the thread of another member.
>> 3) Don't post questions regarding breaking or bypassing any security
>> measure.
>> 4) Acknowledge the responses you receive, good or bad.
>> 5) Jobs posting is not allowed.
>> 6) Sharing copyrighted material and their links is not allowed.
>>
>> NOTE : Don't ever post confidential data in a workbook. Forum owners and
>> members are not responsible for any loss.
>> ---
>> You received this message because you are subscribed to a topic in the
>> Google Groups "MS EXCEL AND VBA MACROS" group.
>> To unsubscribe from this topic, visit
>> https://groups.google.com/d/topic/excel-macros/DrTphcwoTcA/unsubscribe.
>> To unsubscribe from this group and all its topics, send an email to
>> excel-macros+unsubscr...@googlegroups.com.
>> To post to this group, send email to excel-macros@googlegroups.com.
>> Visit this group at http://groups.google.com/group/excel-macros.
>> For more options, visit https://groups.google.com/d/optout.
>>
>
>  --
> Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be? It’s
> =TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @
> https://www.facebook.com/discussexcel
>
> FORUM RULES
>
> 1) Use concise, accurate thread titles. Poor thread titles, like Please
> Help, Urgent, Need Help, Formula Problem, Code Problem, and Need Advice
> will not get quick attention or may not be answered.
> 2) Don't post a question in the thread of another member.
> 3) Don't post questions regarding breaking or bypassing any security
> measure.
> 4) Acknowledge the responses you receive, good or bad.
> 5) Jobs posting is not allowed.
> 6) Sharing copyrighted material and their links is not allowed.
>
> NOTE : Don't ever post confidential data in a workbook. Forum owners and
> members are not responsible for any loss.
> ---
> You received this message because you are subscribed to the Google Groups
> "MS EXCEL AND VBA MACROS" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to excel-macros+unsubscr...@googlegroups.com.
> To post to this group, send email to excel-macros@googlegroups.com.
> Visit this group at http://groups.google.com/group/excel-macros.
> For more options, visit https://groups.google.com/d/optout.
>

-- 
Are you =EXP(E:RT) or =NOT(EXP(E:RT)) in Excel? And do you wanna be? It’s 
=TIME(2,DO:IT,N:OW) ! Join official Facebook page of this forum @ 
https://www.facebook.com/discussexcel

FORUM RULES

1) Use concise, accurate thread titles. Poor thread titles, like Please Help, 
Urgent, Need Help, Formula Problem, Code Problem, and Need Advice will not get 
quick attention or may not be answered.
2) Don't post a question in the thread of another member.
3) Don't post questions regarding breaking or bypassing any security measure.
4) Acknowledge the responses you receive, good or bad.
5) Jobs posting is not allowed.
6) Sharing copyrighted material and their links is not allowed.

NOTE  : Don't ever post confidential data in a workbook. Forum owners and 
members are not responsible for any loss.
--- 
You received this message because you are subscribed to the Google Groups "MS 
EXCEL AND VBA MACROS" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to excel-macros+unsubscr...@googlegroups.com.
To post to this group, send email to excel-macros@googlegroups.com.
Visit this group at http://groups.google.com/group/excel-macros.
For more options, visit https://groups.google.com/d/optout.

Reply via email to