]> git.mxchange.org Git - friendica.git/commitdiff
reload the acl if a new event edit or copy modal is opened
authorrabuzarus <>
Sun, 15 Oct 2017 20:34:31 +0000 (22:34 +0200)
committerrabuzarus <>
Sun, 15 Oct 2017 20:34:31 +0000 (22:34 +0200)
mod/events.php
view/theme/frio/js/event_edit.js
view/theme/frio/js/modal.js
view/theme/frio/templates/event.tpl
view/theme/frio/templates/event_form.tpl

index eec970d0360d1016604fbe1848592e10d20fa59c..5fd093b396ceee5586fc8a4dc8202814630f0243 100644 (file)
@@ -441,7 +441,7 @@ function events_content(App $a) {
                        $sh_checked = (($orig_event['allow_cid'] === '<' . local_user() . '>' && (! $orig_event['allow_gid']) && (! $orig_event['deny_cid']) && (! $orig_event['deny_gid'])) ? '' : ' checked="checked" ');
                }
 
-               if ($cid || ($mode !== 'new')) {
+               if ($cid || $mode === 'edit') {
                        $sh_disabled = 'disabled="disabled"';
                }
 
@@ -474,6 +474,8 @@ function events_content(App $a) {
 
                require_once 'include/acl_selectors.php' ;
 
+               $perms = get_acl_permissions($orig_event);
+
                if ($mode === 'new' || $mode === 'copy') {
                        $acl = (($cid) ? '' : populate_acl(((x($orig_event)) ? $orig_event : $a->user)));
                }
@@ -489,9 +491,14 @@ function events_content(App $a) {
 
                $o .= replace_macros($tpl,array(
                        '$post' => System::baseUrl() . '/events',
-                       '$eid' => $eid,
-                       '$cid' => $cid,
-                       '$uri' => $uri,
+                       '$eid'  => $eid,
+                       '$cid'  => $cid,
+                       '$uri'  => $uri,
+
+                       '$allow_cid' => json_encode($perms['allow_cid']),
+                       '$allow_gid' => json_encode($perms['allow_gid']),
+                       '$deny_cid'  => json_encode($perms['deny_cid']),
+                       '$deny_gid'  => json_encode($perms['deny_gid']),
 
                        '$title' => t('Event details'),
                        '$desc' => t('Starting date and Title are required.'),
index 2eec51916a8b3fab3a416568d018aabeb99b813d..da0837223a4af7754f4dc58a1d5937dab5334bad 100644 (file)
@@ -50,6 +50,24 @@ $(document).ready(function() {
                e.preventDefault();
        });
 
+       // Construct a new ACL. We need this everytime the 'event-edit-form' is loaded
+       // without page reloading (e.g. closing an old modal and open a new modal).
+       // Otherwise we wouldn't get the ACL data.
+       /// @todo: Try to implement some kind of ACL reloading in acl.js.
+       if (typeof acl !== "undefined") {
+               var eventPerms = document.getElementById('event-edit-form');
+
+               acl = new ACL(
+                       baseurl + "/acl",
+                       [
+                               eventPerms.dataset.allow_cid,
+                               eventPerms.dataset.allow_gid,
+                               eventPerms.dataset.deny_cid,
+                               eventPerms.dataset.deny_gid
+                       ]
+               );
+       }
+
 });
 
 // Load the html of the actual event and incect the output to the
@@ -63,19 +81,6 @@ function doEventPreview() {
 }
 
 
-// This function load the content of the edit url into a modal.
-function eventEdit(url) {
-       var char = qOrAmp(url);
-       url = url + char + 'mode=none';
-
-       $.get(url, function(data) {
-               $("#modal-body").empty();
-               $("#modal-body").append(data);
-       }).done(function() {
-               loadModalTitle();
-       });
-}
-
 // The following functions show/hide the specific event-edit content 
 // in dependence of the selected nav.
 function eventAclActive() {
index 1fd03ddfac54585fec7e0ba01eccf278152acc13..2b60049f1d4a37392cbd3e15f35e7a00939fc67b 100644 (file)
@@ -1,39 +1,39 @@
-/* 
- * @brief contains functions for bootstrap modal handling
+/**
+ * @brief Contains functions for bootstrap modal handling.
  */
 $(document).ready(function(){
-       // Clear bs modal on close
-       // We need this to prevent that the modal displays old content
+       // Clear bs modal on close.
+       // We need this to prevent that the modal displays old content.
        $('body, footer').on('hidden.bs.modal', '.modal', function () {
                $(this).removeData('bs.modal');
                $("#modal-title").empty();
                $('#modal-body').empty();
                // Remove the file browser from jot (else we would have problems
-               // with AjaxUpload
+               // with AjaxUpload.
                $(".fbrowser").remove();
-               // Remove the AjaxUpload element
+               // Remove the AjaxUpload element.
                $("[name=userfile]").parent().remove();
        });
 
-       // Clear bs modal on close
-       // We need this to prevent that the modal displays old content
+       // Clear bs modal on close.
+       // We need this to prevent that the modal displays old content.
        $('body').on('hidden.bs.modal', '#jot-modal', function () {
-               // restore cached jot at its hidden position ("#jot-content")
+               // Restore cached jot at its hidden position ("#jot-content").
                $("#jot-content").append(jotcache);
-               // clear the jotcache
+               // Clear the jotcache.
                jotcache = '';
        });
 
-       // Add Colorbox for viewing Network page images
+       // Add Colorbox for viewing Network page images.
        //var cBoxClasses = new Array();
        $("body").on("click", ".wall-item-body a img", function(){
                var aElem = $(this).parent();
                var imgHref = aElem.attr("href");
 
-               // We need to make sure we only put a Colorbox on links to Friendica images
+               // We need to make sure we only put a Colorbox on links to Friendica images.
                // We'll try to do this by looking for links of the form
                // .../photo/ab803d8eg08daf85023adfec08 (with nothing more following), in hopes
-               // that that will be unique enough
+               // that that will be unique enough.
                if(imgHref.match(/\/photo\/[a-fA-F0-9]+(-[0-9]\.[\w]+?)?$/)) {
 
                        // Add a unique class to all the images of a certain post, to allow scrolling through
@@ -46,52 +46,52 @@ $(document).ready(function(){
 
                        aElem.colorbox({
                                maxHeight: '90%',
-                               photo: true, // Colorbox doesn't recognize a URL that don't end in .jpg, etc. as a photo
-                               rel: cBoxClass //$(this).attr("class").match(/wall-item-body-[\d]+-lightbox/)[0]
+                               photo: true, // Colorbox doesn't recognize a URL that don't end in .jpg, etc. as a photo.
+                               rel: cBoxClass //$(this).attr("class").match(/wall-item-body-[\d]+-lightbox/)[0].
                        });
                }
        });
 
-       // Navbar login
+       // Navbar login.
        $("body").on("click", "#nav-login", function(e){
                e.preventDefault();
                Dialog.show(this.href, this.dataset.originalTitle || this.title);
        });
 
-       // Jot nav menu.
+       // Jot nav menu..
        $("body").on("click", "#jot-modal .jot-nav li .jot-nav-lnk", function(e){
                e.preventDefault();
                toggleJotNav(this);
        });
 
-       // bookmarklet page needs an jot modal which appears automatically
+       // Bookmarklet page needs an jot modal which appears automatically.
        if(window.location.pathname.indexOf("/bookmarklet") >=0 && $("#jot-modal").length){
                jotShow();
        }
 
        // Open filebrowser for elements with the class "image-select"
-       // The following part handles the filebrowser for field_fileinput.tpl
+       // The following part handles the filebrowser for field_fileinput.tpl.
        $("body").on("click", ".image-select", function(){
-               // set a extra attribute to mark the clicked button
+               // Set a extra attribute to mark the clicked button.
                this.setAttribute("image-input", "select");
                Dialog.doImageBrowser("input");
        });
 
-       // Insert filebrowser images into the input field (field_fileinput.tpl)
+       // Insert filebrowser images into the input field (field_fileinput.tpl).
        $("body").on("fbrowser.image.input", function(e, filename, embedcode, id, img) {
-               // select the clicked button by it's attribute
+               // Select the clicked button by it's attribute.
                var elm = $("[image-input='select']");
-               // select the input field which belongs to this button
+               // Select the input field which belongs to this button.
                var input = elm.parent(".input-group").children("input");
-               // remove the special indicator attribut from the button
+               // Remove the special indicator attribut from the button.
                elm.removeAttr("image-input");
-               // inserte the link from the image into the input field
+               // Insert the link from the image into the input field.
                input.val(img);
                
        });
 });
 
-// overwrite Dialog.show from main js to load the filebrowser into a bs modal
+// Overwrite Dialog.show from main js to load the filebrowser into a bs modal.
 Dialog.show = function(url, title) {
        if (typeof(title) === 'undefined') {
                title = "";
@@ -112,21 +112,21 @@ Dialog.show = function(url, title) {
                });
 };
 
-// overwrite the function _get_url from main.js
+// Overwrite the function _get_url from main.js.
 Dialog._get_url = function(type, name, id) {
        var hash = name;
        if (id !== undefined) hash = hash + "-" + id;
        return "fbrowser/"+type+"/?mode=none#"+hash;
 };
 
-// does load the filebrowser into the jot modal
+// Does load the filebrowser into the jot modal.
 Dialog.showJot = function() {
        var type = "image";
        var name = "main";
 
        var url = Dialog._get_url(type, name);
        if(($(".modal-body #jot-fbrowser-wrapper .fbrowser").length) < 1 ) {
-               // load new content to fbrowser window
+               // Load new content to fbrowser window.
                $("#jot-fbrowser-wrapper").load(url,function(responseText, textStatus){
                        if ( textStatus === 'success' || 
                                textStatus === 'notmodified') 
@@ -137,18 +137,18 @@ Dialog.showJot = function() {
        }
 };
 
-// init the filebrowser after page load
+// Init the filebrowser after page load.
 Dialog._load = function(url) {
-       // get nickname & filebrowser type from the modal content
+       // Get nickname & filebrowser type from the modal content.
        var nickname = $("#fb-nickname").attr("value");
        var type = $("#fb-type").attr("value");
 
-       // try to fetch the hash form the url
+       // Try to fetch the hash form the url.
        var match = url.match(/fbrowser\/[a-z]+\/\?mode=none(.*)/);
        if (match===null) return; //not fbrowser
        var hash = match[1];
 
-       // initialize the filebrowser
+       // Initialize the filebrowser.
        var jsbrowser = function() {
                FileBrowser.init(nickname, type, hash);
        };
@@ -163,15 +163,15 @@ Dialog._load = function(url) {
  * moment or where it isn't possible because of design
  */
 function loadModalTitle() {
-       // clear the text of the title
+       // Clear the text of the title.
        $("#modal-title").empty();
 
-       // hide the first element with the class "heading" of the modal body
+       // Hide the first element with the class "heading" of the modal body.
        $("#modal-body .heading").first().hide();
 
        var title = "";
 
-       // get the text of the first element with "heading" class
+       // Get the text of the first element with "heading" class.
        title = $("#modal-body .heading").first().text();
 
        // for event modals we need some speacial handling
@@ -181,14 +181,14 @@ function loadModalTitle() {
                title = title + eventsum;
        }
 
-       // and append it to modal title
+       // And append it to modal title.
        if (title!=="") {
                $("#modal-title").append(title);
        }
 }
 
 // This function loads html content from a friendica page
-// into a modal
+// into a modal.
 function addToModal(url) {
        var char = qOrAmp(url);
 
@@ -204,13 +204,13 @@ function addToModal(url) {
                                modal.show();
 
                                //Get first element with the class "heading"
-                               //and use it as title
+                               //and use it as title.
                                loadModalTitle();
                        }
                });
 }
 
-// Add a element (by it's id) to a bootstrap modal
+// Add a element (by it's id) to a bootstrap modal.
 function addElmToModal(id) {
        var elm = $(id).html();
        var modal = $('#modal').modal();
@@ -223,17 +223,17 @@ function addElmToModal(id) {
        loadModalTitle();
 }
 
-// function to load the html from the edit post page into
-// the jot modal
+// Function to load the html from the edit post page into
+// the jot modal.
 function editpost(url) {
-       // next to normel posts the post can be an event post. The event posts don't
+       // Next to normel posts the post can be an event post. The event posts don't
        // use the normal Jot modal. For event posts we will use a normal modal
        // But first we have to test if the url links to an event. So we will split up
-       // the url in its parts
+       // the url in its parts.
        var splitURL = parseUrl(url);
        // Test if in the url path containing "events/event". If the path containing this
        // expression then we will call the addToModal function and exit this function at
-       // this point
+       // this point.
        if (splitURL.path.indexOf('events/event') > -1) {
                addToModal(splitURL.path);
                return;
@@ -250,11 +250,11 @@ function editpost(url) {
        // After closing the modal original jot should be restored at its orginal position in the html structure.
        jotcache = $("#jot-content > #jot-sections");
 
-       // remove the original Jot as long as the edit Jot is open
+       // Remove the original Jot as long as the edit Jot is open.
        jotcache.remove();
 
-       // add the class "edit" to the modal to have some kind of identifier to
-       // have the possibility to e.g. put special event-listener
+       // Add the class "edit" to the modal to have some kind of identifier to
+       // have the possibility to e.g. put special event-listener.
        $("#jot-modal").addClass("edit-jot");
 
        jotreset();
@@ -265,11 +265,11 @@ function editpost(url) {
                        if ( textStatus === 'success' || 
                                textStatus === 'notmodified') 
                        {
-                               // get the item type and hide the input for title and category if it isn't needed
+                               // get the item type and hide the input for title and category if it isn't needed.
                                var type = $(responseText).find("#profile-jot-form input[name='type']").val();
                                if(type === "wall-comment" || type === "remote-comment")
                                {
-                                       // hide title and category input fields because we don't
+                                       // Hide title and category input fields because we don't.
                                        $("#profile-jot-form #jot-title-wrap").hide();
                                        $("#profile-jot-form #jot-category-wrap").hide();
                                }
@@ -280,10 +280,10 @@ function editpost(url) {
                });
 }
 
-// remove content from the jot modal
+// Remove content from the jot modal.
 function jotreset() {
-       // Clear bs modal on close
-       // We need this to prevent that the modal displays old content
+       // Clear bs modal on close.
+       // We need this to prevent that the modal displays old content.
        $('body').on('hidden.bs.modal', '#jot-modal.edit-jot', function () {
                $(this).removeData('bs.modal');
                $(".jot-nav .jot-perms-lnk").parent("li").removeClass("hidden");
@@ -291,32 +291,32 @@ function jotreset() {
                $("#profile-jot-form #jot-category-wrap").show();
 
                // the following was commented out because it is needed anymore
-               // because we changed the behavior at an other place
+               // because we changed the behavior at an other place.
        //              var rand_num = random_digits(12);
        //              $('#jot-title, #jot-category, #profile-jot-text').val("");
        //              $( "#profile-jot-form input[name='type']" ).val("wall");
        //              $( "#profile-jot-form input[name='post_id']" ).val("");
        //              $( "#profile-jot-form input[name='post_id_random']" ).val(rand_num);
 
-               // remove the "edit-jot" class so we can the standard behavior on close
+               // Remove the "edit-jot" class so we can the standard behavior on close.
                $("#jot-modal.edit-jot").removeClass("edit-jot");
                $("#jot-modal-content").empty();
        });
 }
 
-// Give the active "jot-nav" list element the class "active"
+// Give the active "jot-nav" list element the class "active".
 function toggleJotNav (elm) {
-       // Get the ID of the tab panel which should be activated
+       // Get the ID of the tab panel which should be activated.
        var tabpanel = elm.getAttribute("aria-controls");
        var cls = hasClass(elm, "jot-nav-lnk-mobile");
 
-       // Select all li of jot-nav and remove the active class
+       // Select all li of jot-nav and remove the active class.
        $(elm).parent("li").siblings("li").removeClass("active");
-       // Add the active class to the parent of the link which was selected
+       // Add the active class to the parent of the link which was selected.
        $(elm).parent("li").addClass("active");
 
        // Minimize all tab content wrapper and activate only the selected
-       // tab panel
+       // tab panel.
        $('#jot-modal [role=tabpanel]').addClass("minimize").attr("aria-hidden" ,"true");
        $('#jot-modal #' + tabpanel).removeClass("minimize").attr("aria-hidden" ,"false");
 
@@ -324,7 +324,7 @@ function toggleJotNav (elm) {
        $("#jot-modal .nav-tabs .jot-nav-lnk").attr("aria-selected", "false");
        elm.setAttribute("aria-selected", "true");
 
-       // For some some tab panels we need to execute other js functions
+       // For some some tab panels we need to execute other js functions.
        if (tabpanel === "jot-preview-content") {
                preview_post();
        } else if (tabpanel === "jot-fbrowser-wrapper") {
@@ -333,7 +333,7 @@ function toggleJotNav (elm) {
                });
        }
 
-       // If element is a mobile dropdown nav menu we need to change the botton text
+       // If element is a mobile dropdown nav menu we need to change the botton text.
        if (cls) {
                toggleDropdownText(elm);
        }
@@ -341,17 +341,31 @@ function toggleJotNav (elm) {
 
 // Wall Message needs a special handling because in some cases
 // it redirects you to your own server. In such cases we can't
-// load it into a modal
+// load it into a modal.
 function openWallMessage(url) {
-       // split the the url in its parts
+       // Split the the url in its parts.
        var parts = parseUrl(url);
 
        // If the host isn't the same we can't load it in a modal.
-       // So we will go to to the url directly
+       // So we will go to to the url directly.
        if( ("host" in parts) && (parts.host !== window.location.host)) {
                window.location.href = url;
        } else {
-               // otherwise load the wall message into a modal
+               // Otherwise load the wall message into a modal.
                addToModal(url);
        }
 }
+
+// This function load the content of the edit url into a modal.
+/// @todo Rename this function because it can be used for more than events.
+function eventEdit(url) {
+       var char = qOrAmp(url);
+       url = url + char + 'mode=none';
+
+       $.get(url, function(data) {
+               $("#modal-body").empty();
+               $("#modal-body").append(data);
+       }).done(function() {
+               loadModalTitle();
+       });
+}
index ed3877665e2cfa12b0b5e29ca052de20b57fb872..947d09cb8d97828c6175e32ebfebc1e3817c2c3f 100644 (file)
@@ -1,5 +1,3 @@
-<script language="javascript" type="text/javascript"
-         src="{{$baseurl}}/view/theme/frio/js/event_edit.js"></script>
 
 {{foreach $events as $event}}
        <div class="event-wrapper">
index c34fadc8ad9aef8176b2bd976b559190027555e9..e1012e3ae8f12d3ea63633acf555b4476b3ad941 100644 (file)
@@ -29,7 +29,7 @@
        </ul>
 
        <div id="event-edit-form-wrapper">
-       <form id="event-edit-form" action="{{$post}}" method="post" >
+       <form id="event-edit-form" action="{{$post}}" method="post" data-allow_cid="{{$allow_cid}}" data-allow_gid="{{$allow_gid}}" data-deny_cid="{{$deny_cid}}" data-deny_gid="{{$deny_gid}}">
 
                <input type="hidden" name="event_id" value="{{$eid}}" />
                <input type="hidden" name="cid" value="{{$cid}}" />