Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't Delete Portals in DNN 9.2.1 #2351

Closed
1 of 8 tasks
jncraig opened this issue Oct 1, 2018 · 5 comments · Fixed by #2701
Closed
1 of 8 tasks

Can't Delete Portals in DNN 9.2.1 #2351

jncraig opened this issue Oct 1, 2018 · 5 comments · Fixed by #2701
Milestone

Comments

@jncraig
Copy link

jncraig commented Oct 1, 2018

Description of bug

Deleting a portal (Persona Bar "Sites") doesn't work.

Steps to reproduce

Go to Persona Bar, Manage, Sites.
Mouseover a site and click the trashcan icon to delete.

Current result

A generic error pops up, nothing is logged in the Event Log, but an error message appears in the xxx.log.resources file:

2018-10-01 11:17:37.780-04:00 [Surface-Joe][D:2][T:15][ERROR] Dnn.PersonaBar.Sites.Services.SitesController - System.UnauthorizedAccessException: Access to 'C:\Users\Joe\Documents\Websites\DNN921-test\Website' denied. ---> System.ComponentModel.Win32Exception: Access is denied
--- End of inner exception stack trace ---
at SchwabenCode.QuickIO.Internal.InternalQuickIOCommon.NativeExceptionMapping(String path, Int32 errorCode)
at SchwabenCode.QuickIO.Internal.InternalQuickIO.GetFindDataFromPath(QuickIOPathInfo pathInfo)
at DotNetNuke.Common.Utilities.FileSystemUtils.DeleteFilesRecursive(String strRoot, String filter)
at DotNetNuke.Entities.Portals.PortalController.DeletePortal(PortalInfo portal, String serverPath)
at Dnn.PersonaBar.Sites.Services.SitesController.DeletePortal(Int32 portalId)

Expected result

Site is deleted

Screenshots

If applicable, provide screenshots to help explain your problem.

Error log

There is no Event Log entry.
The error in xx.log.resources is:

the xxx.log.resources file:

2018-10-01 11:17:37.780-04:00 [Surface-Joe][D:2][T:15][ERROR] Dnn.PersonaBar.Sites.Services.SitesController - System.UnauthorizedAccessException: Access to 'C:\Users\Joe\Documents\Websites\DNN921-test\Website' denied. ---> System.ComponentModel.Win32Exception: Access is denied
--- End of inner exception stack trace ---
at SchwabenCode.QuickIO.Internal.InternalQuickIOCommon.NativeExceptionMapping(String path, Int32 errorCode)
at SchwabenCode.QuickIO.Internal.InternalQuickIO.GetFindDataFromPath(QuickIOPathInfo pathInfo)
at DotNetNuke.Common.Utilities.FileSystemUtils.DeleteFilesRecursive(String strRoot, String filter)
at DotNetNuke.Entities.Portals.PortalController.DeletePortal(PortalInfo portal, String serverPath)
at Dnn.PersonaBar.Sites.Services.SitesController.DeletePortal(Int32 portalId)

Additional context

Instead, I get a generic error box popping up over the Personal Bar, nothing in the Event Log, but this in

I can delete portals in 8.0.4.

In poking around, deleting a portal (8.0.4) doesn't delete a lot of stuff in most of the portal related tables. I think that it should ...

Affected version

  • [] 9.2.2
  • 9.2.1
  • [] 9.2
  • [] 9.1.1
  • 9.1
  • 9.0

Affected browser

N/A

  • Chrome
  • Firefox
  • Safari
  • Internet Explorer
  • Edge
@jncraig
Copy link
Author

jncraig commented Oct 3, 2018

This issue is present in 9.2.2

@ohine ohine changed the title Can't Delete Portals in DNN 9..2.1 Can't Delete Portals in DNN 9.2.1 Oct 4, 2018
@ohine ohine added the Type: Bug label Oct 4, 2018
@sleupold
Copy link
Contributor

sleupold commented Oct 7, 2018

has this been tested in DNN 9.2.2 as well?

@digibum
Copy link

digibum commented Feb 13, 2019

Same error in v. 09.02.02 (178)

@valadas
Copy link
Contributor

valadas commented Sep 9, 2019

This PR was mistakenly merged into development instead of release/9.4.x and then release/9.4.x was merged into development, thus reverting this fix by mistake.

@zyhfish will resubmit this targetting release/9.4.x soon

@valadas
Copy link
Contributor

valadas commented Sep 10, 2019

Merged

@valadas valadas closed this as completed Sep 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants