Copied to clipboard

Flag this post as spam?

This post will be reported to the moderators as potential spam to be looked at


  • Tim Geyssens 6562 posts 15373 karma points MVP 2x c-trib
    Sep 08, 2016 @ 10:30
    Tim Geyssens
    0

    Prop editor doesn't seem to respect user start forms folder

    Hey great job on the package, just tested and it seems like the prop editor outputs all folders and forms and doesn't take the user start folder into account...

  • [email protected] 378 posts 1915 karma points MVP 4x c-trib
    Sep 08, 2016 @ 12:05
    jeffrey@umarketingsuite.com
    0

    Hi Tim,

    thanks for your kuddoos! Can you send a picture about what you mean?

    Thanks, Jeffrey

  • Tim Geyssens 6562 posts 15373 karma points MVP 2x c-trib
    Sep 08, 2016 @ 12:38
    Tim Geyssens
    0

    Sure thing,

    So this is how the entire forms tree looks

    enter image description here

    The user has been restricted to 'Folder' and after that only sees this in the Forms tree

    enter image description here

    But when using the form picker prop editor he can see the entire tree and forms collection

    enter image description here

    Looking at the code for the prop editor it doesn't seem to take it into account...

  • [email protected] 378 posts 1915 karma points MVP 4x c-trib
    Sep 08, 2016 @ 14:26
    jeffrey@umarketingsuite.com
    0

    Hi Tim,

    thanks for the pictures. That explains a lot.

    When we designed this functionality we wanted to restrict only the nodes in the Forms-section, so you couldn't edit the nodes you didn't have access to.

    Put inserting wouldn't be a problem because maybe you couldn't edit form X (because in our use case there was an editor that did most of the editing of the forms) but you could insert it into specific pages.

    So it sort of by design, but I see your point. Wondering what the best way is to fix this? Any suggestions,

    Jeffrey

    p.s.: GitHub will be updated soon :)

  • Tim Geyssens 6562 posts 15373 karma points MVP 2x c-trib
    Sep 08, 2016 @ 14:34
    Tim Geyssens
    0

    Well you could make it a config option on the prop editor... , simple checkbox if that is checked then make sure to check the user folder permission

    Since in our case we are running a multi site Umbraco instance where user x shouldn't have any access to an other site's forms (each site will have its forms in a separate folder)

  • Tim Geyssens 6562 posts 15373 karma points MVP 2x c-trib
    Sep 08, 2016 @ 12:42
    Tim Geyssens
    0

    Ow and on a side not code on github seems to be outdated... (but again thanks for sharing :) )

  • Timo Perplex 13 posts 140 karma points c-trib
    Oct 05, 2016 @ 12:03
    Timo Perplex
    1

    Hi Tim,

    I've just updated the Github source code to include the latest changes. Should be all up-to-date now.

    https://github.com/PerplexInternetmarketing/Perplex-Umbraco-Forms/

  • Tim Geyssens 6562 posts 15373 karma points MVP 2x c-trib
    Oct 05, 2016 @ 12:35
    Tim Geyssens
    0

    Awesome, thanks Timo!

  • Tom van Enckevort 100 posts 387 karma points
    Jun 21, 2017 @ 15:25
    Tom van Enckevort
    0

    Was this configuration option ever implemented?

    We have a very similar scenario where a multi-site Umbraco install has several users who all need to be restricted to their own set of forms, so it would be nice to be able to limit the form picker list to the start node(s) of the current user.

  • [email protected] 378 posts 1915 karma points MVP 4x c-trib
    Jun 21, 2017 @ 15:27
    jeffrey@umarketingsuite.com
    1

    Hi Tom,

    if you download the latest version of the package this can be done.

    Cheerio, Jeffrey

  • Tom van Enckevort 100 posts 387 karma points
    Jun 22, 2017 @ 07:43
    Tom van Enckevort
    0

    Hi Jeffrey,

    Just looking at it in v1.7, but cannot seem to find an option for this? Nor can I see anything that has changed in the code to add it in?

  • This forum is in read-only mode while we transition to the new forum.

    You can continue this topic on the new forum by tapping the "Continue discussion" link below.

Please Sign in or register to post replies