Merge pull request #26 from valadas/Issue4

Fixed an issue where setting Use Permissions would not apply
DNNCommunity · Jul 14, 2018 · 7b35154 · 7b35154
2 parents be33130 + b1f5426
commit 7b35154
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/Links.ascx.cs b/Links.ascx.cs
@@ -748,7 +748,7 @@ private void Page_Load(System.Object sender, System.EventArgs e)
                             if (this.ModuleContentType == Enums.ModuleContentTypes.Friends)
                                 isInARole = true;
 
-                            if (isInARole | link.GrantRoles.Contains(";-2;") | link.GrantRoles.Contains(";-1;") | this.UserInfo.IsSuperUser | this.UserInfo.IsInRole(this.PortalSettings.AdministratorRoleName))
+                            if (!this.UsePermissions | isInARole | link.GrantRoles.Contains(";-2;") | link.GrantRoles.Contains(";-1;") | this.UserInfo.IsSuperUser | this.UserInfo.IsInRole(this.PortalSettings.AdministratorRoleName))
 
                                 linksToShow.Add(link);
                         }

diff --git a/ReleaseNotes.htm b/ReleaseNotes.htm
@@ -5,6 +5,9 @@ <h3>Bug fixes</h3>
     <li>
         Fixed build dependencies issues
     </li>
+    <li>
+        Fixed an issue where setting Use Permissions to No would have no effect after it was set to Yes.
+    </li>
 </ul>
 <hr />
-Original file line number
+Diff line change
@@ Expand Up / @@ -5,6 +5,9 @@ <h3>Bug fixes</h3> @@
         <li>
             Fixed build dependencies issues
         </li>
+        <li>
+            Fixed an issue where setting Use Permissions to No would have no effect after it was set to Yes.
+        </li>
     </ul>
     <hr />
@@ Expand Down @@