Re: [phpMyAdmin Developers] Adding drag and drop disable feature

Isaac Bennetch <[email protected]>
Newsgroups gmane.comp.php.phpmyadmin.devel
Message-ID <CAFb5Y7qYvNaROqJaggYQoibE=pdZJHp+rYgX+gdULWochqXHqw@mail.gmail.com>
On Sat, Jun 23, 2018 at 6:58 AM, Lakshay arora
<[email protected]> wrote:
> Hello Developers,
> I'm currently working on this feature as a part of my GSoC project. Isaac
> and I have been trying to think of some workarounds but your ideas are
> welcome.
> I have currently thought of the following workarounds.
>
> Disable drag drop at the backend, but drag drop handlers in javascript are
> written in common,js which has other stuff also, so we can either separate
> it in some other file(which I don't think would be a good solution because
> it might cause some bugs), or we can find a way to expose the configuration
> to javascript(not sure but could lead to security issues) or we can leave
> the javascript as it is and disable it only on the server side.(it still
> works well, I've tested).

I have a few (untested) ideas about this, although I can't think of
anything else in the code where a JavaScript configuration is
controlled by a config.inc.php directive.

Can we move the part where we attach the event in js/common.js to a
PHP file? If we can make these lines conditional that would prevent
the attachment in the first place
> common.js:$(document).on('dragenter', PMA_DROP_IMPORT._dragenter);
> common.js:$(document).on('dragover', PMA_DROP_IMPORT._dragover);
> common.js:$(document).on('dragleave', '.pma_drop_handler', PMA_DROP_IMPORT._dragleave);
> common.js:$(document).on('drop', 'body', PMA_DROP_IMPORT._drop);

Can we move the PMA_DROP_IMPORT functionality to a new file
(js/drag_drop_import.js) and conditionally include that, like we do
with CodemirrorEnable and libraries/classes/Header.php?

Can we set a variable early on in the execution of the common PHP
functions that are called on every page where we set a JavaScript
variable, then perform the test in JavaScript?

This is a new area for me, so those are a few of my early-morning ideas.


> Thanks,
> Lakshya arora (nulll-pointer)
>
>
> _______________________________________________
> Developers mailing list
> [email protected]
> https://lists.phpmyadmin.net/mailman/listinfo/developers
>

_______________________________________________
Developers mailing list
[email protected]
https://lists.phpmyadmin.net/mailman/listinfo/developers
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.