Hi Dave,
The above error was not thrown when import 'jquery.event.drag' was added to
column_selector_spec.js (although it was giving some other error, but above
was cleared).
So the doubt was on webpack as it was not importing jquery.event.drag even
though it was added using imports-loader, the below test:
test: /.*slickgrid\/+slick\.(?!core)*/,
loader: 'imports-loader?' +
'jquery.ui' +
',jquery.event.drag' +
',slickgrid',
I noticed later that the test regex is not just for files but also included
path. So, gave try by replacing "\/" with "\\" and it worked.
But then this will not work on Unix, and so changed it to [\\\/]+ taking
both styles.
Thanks and Regards,
Aditya Toshniwal
Software Engineer | EnterpriseDB Software Solutions | Pune
"Don't Complain about Heat, Plant a tree"
On Fri, May 11, 2018 at 3:05 PM, Dave Page <dp...@pgadmin.org> wrote:
>
>
> On Fri, May 11, 2018 at 7:28 AM, Aditya Toshniwal <aditya.toshniwal@
> enterprisedb.com> wrote:
>
>> Hi Dave/Hackers,
>>
>> PFA fix for the problem. The RegEx for webpack test is not working on
>> windows because the path separator is different for Windows( \ ) and
>> Unix( / ). And so, the imports loader is not executing for slickgrid.
>> Patch tested on Win 10, Win 7, Ubuntu, Cent OS and works fine.
>>
>
> Nice catch! How did you figure that one out?
>
> Patch applied.
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
