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

Redirect after login not working if Login page has a different name than login #3661

Closed
1 task done
edisfera opened this issue Apr 3, 2020 · 3 comments · Fixed by #4171
Closed
1 task done

Redirect after login not working if Login page has a different name than login #3661

edisfera opened this issue Apr 3, 2020 · 3 comments · Fixed by #4171

Comments

@edisfera
Copy link

edisfera commented Apr 3, 2020

Description of bug

If a page to be redirected after login is set in Settings > Security tab Login settings but the login page name is called different from Login (i.e. Enter) the redirect page is ignored.

Steps to reproduce

  1. Create a login page not named Login (buy the way this is trapped by DNN and one cannot create a page called Login!) and insert the login module in it. Then set that page to be the login page of the portal.
  2. Set a page to be redirected to after login
  3. Then try to login

Current behavior

The user is redirected to the home page instead of the intended page (the redirect after login page)

Expected behavior

The user should be directed to the redirect after login page

Error information

N/A

Additional context

The error is caused by the following function in this source file Dnn.Platform-9.5.0\DNN Platform\Website\DesktopModules\Admin\Authentication\Login.ascx.cs at line 256:

private bool IsRedirectingFromLoginUrl()
{
return Request.UrlReferrer != null &&
Request.UrlReferrer.LocalPath.ToLowerInvariant().EndsWith(LOGIN_PATH);
}
because LOGIN_PATH is declared as follows
private const string LOGIN_PATH = "/login";
If the page is called Enter the LocalPath will be something like "/en-Us/Enter" and of course will never end with "/Login" thus returning always false if the login page is not called Login.

Affected version

  • 9.5.0 latest supported release
@edisfera
Copy link
Author

edisfera commented Apr 3, 2020

a possible fix could be the following (don't forget to comment out the declaration of LOGIN_PATH that is no longer needed):
private string GetLoginPath()
{
if (PortalSettings.LoginTabId == Null.NullInteger)
{
return "/login";
}
else
{
var tab = TabController.Instance.GetTab(PortalSettings.LoginTabId, PortalId);
return tab != null ? "/" + tab.TabName : "/login";
}
}

    private bool IsRedirectingFromLoginUrl()
    {
        return Request.UrlReferrer != null &&
            Request.UrlReferrer.LocalPath.ToLowerInvariant().EndsWith(GetLoginPath().ToLowerInvariant());
    }

@david-poindexter david-poindexter added this to Awaiting Triage in Issue Triage via automation May 6, 2020
@moorecreative
Copy link

I believe we are experiencing the same thing, however, the redirect does not work in 9.5 (where it did work before 9.4 and earlier) regardless of the Login page name/type.

In our scenario,

  • the Login page is called "Login" and is a page within DNN pages,
  • this page is set as the Login page within Site Settings
  • if you call a page which has roles requiring Registered Users for view permission, requesting the page loads correctly sitename.com/Login?returnurl=%2fName-Of-Page
  • however, after successful login, it remains on the /Login page and does not redirect to the ReturnURL path specified in the query string.

@kmuralidaran
Copy link
Contributor

@edisfera I am not sure whether my below observation is simplified version of reported bug or another bug

  1. Login to DNN
  2. Set a page to be redirected to after login (PostLogin)
  3. Navigate to a page (Page1)
  4. Logout
  5. Login to DNN from current page (Page1)

Current behavior : Current page retained and no redirection happened, Redirection to PostLogin happens only if the current page is the home page

Expected behavior : The user should be redirected to the PostLogin page

kmuralidaran added a commit to kmuralidaran/Dnn.Platform that referenced this issue Oct 13, 2020
Issue Triage automation moved this from Bugs to Closed Oct 15, 2020
bdukes pushed a commit that referenced this issue Oct 15, 2020
mtrutledge added a commit to mtrutledge/Dnn.Platform that referenced this issue Dec 18, 2020
* Changed accessor of ISerialzationManager property to private

* Updated startup configuration to use scoped instead of transient. Also updated typo in ISerializationManager registration

* Resolves an infinite redirect loop issue

Closes dnnsoftware#4157

An infinite redirect loop could occur even in situations where the portal aliasses where setup in canonical mode.

Relates to this discussion: dnnsoftware#4137

* Update DNN Platform/Library/Entities/Modules/Settings/SettingsRepository.cs

* Properly named CSS class & keeps the other for compatibility for now (dnnsoftware#4160)

* Adds IEventLogger to Dependency Injection (dnnsoftware#4159)

* Extracted IEventLogger, IEventLogService, and IEventLogConfigService to Abstractions project
* Updated ILogProperties to inherit from ICollection<ILogInfoDetail>
* Fixed property name casing to match new abstractions standard
* Updated new API to return IEnumerable<ILogInfo> instead of ArrayList
* IEventLogService implements IEventLogger
* Deprecated GetSingleLog and added a new API GetLog
* Move AddSettingLog to an extension method

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Adds support for editing the robots.txt on a website (dnnsoftware#4165)

* Adds support for editing the robots.txt on a website

Resolves dnnsoftware#3103

* Moved robots.txt get/update methods & made private

* Applied suggested updates from code review

* Swap CodeMirror in SqlConsole for monaco-editor

* Issues/fix pr 4165 (dnnsoftware#4172)

* Fixes testing issue found in PR dnnsoftware#4165

* Fixing the save method too...

* Correctly using Path.Combine

* Swap CodeMirror in ConfigConsole for monaco-editor

* Fix MSBuild issue with Dnn.PersonaBar.Extensions

* Lowercases the meta tag names (dnnsoftware#4176)

Resolves dnnsoftware#2547

* Fixes Event Log from Failing Silently during Application Startup (dnnsoftware#4178)

* Added try-catch block to prevent event logs from silently failing

* Updated exception handling to catch specific exception

* Updated DDR Menu to be System (dnnsoftware#4180)

* Marked all public WebSlice members as obsolete, Part of solution for dnnsoftware#3703 (dnnsoftware#4182)

* Updated the UpdateDatabase procedure to set version numbers (dnnsoftware#4183)

* Updated the UpdateDatabase procedure to set version numbers

* Fixed typo

* Updated order of items for better readability and removed incorrect items.

* Updated Converter for TabSettings Display (dnnsoftware#4184)

* Parse date with invariant culture to fix non-Gregorian calendar dates… (dnnsoftware#4164)

* Parse date with invariant culture to fix non-Gregorian calendar dates issue

* review suggestions incorporated.

* Enable dependency injection for IServiceRouteMapper (dnnsoftware#4155)

* Fixes Social Messaging Ignoring Line Breaks & Copyright Year dnnsoftware#4166 (dnnsoftware#4186)

* Adds token support to replace the hard-coded 2013 copyright date

* Allows line breaks to be displayed as entered by end-users

* Allow line breaks but properly encode all other HTML

Resolves dnnsoftware#4166

* Use new MailAttachment class for MailProvider

In order to support multiple mail providers, this introduces a new
MailAttachment class for use by the MailProvider API. This decoupling from
System.Net.Mail.Attachment will give more flexibility when implementing new
providers, such as MailKit.

* Use ICollection<MailAttachment> instead of List<T>

* Updated Release Schedule Documentation (dnnsoftware#4188)

* Updated Release Schedule Documentation

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Updated release verbiage

* Updated Notes

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Update .github/RELEASE_SCHEDULE.md

* Update .github/RELEASE_SCHEDULE.md

* Update .github/RELEASE_SCHEDULE.md

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>
Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>
Co-authored-by: Daniel Valadas <info@danielvaladas.com>

* Updates versions as per release candidate creation

* Update bug-report.md

* Resolves multiple stylcop warnings in Library and HttpModules projects

* child page without a parent should not be validated

* UnObsoleted IsHostMenu

* Deprecated PortalSettings in WebControlBase

* Deprecated PortalSettings in WebControlBase

* Disables waring on usage of GetCurrentPortalSettings

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Fix tooltip Sitemap page priority

* Cleaned up UserProfileClass

- Resolves many stylecop warnings in UserProfile
- Makes constants for retrieving profile properties obsolute and re-implements them as private constants
- Updated some upstream classes to use IPortalSettings instead of PortalSettings
- Implemented automated stylecop fixes on many other files in the same project

* Deprecated CheckAuditLevel that takes PortalSettings in favour of new one with IPortalSettings

* Update DNN Platform/Library/Services/Tokens/PropertyAccess/IPropertyAccess.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Services/Tokens/PropertyAccess/IPropertyAccess.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Adjusted deprecation comments as suggested

* Reset redirect url for redirecting to 'Redirect After Login' page

* dnnsoftware#3661 - Addressed review comments

* Revert "Merge branch 'bugfix/DNN-44083' of https://github.com/kmuralidaran/Dnn.Platform into bugfix/DNN-44083"

This reverts commit 02fb399, reversing
changes made to b78f1b8.

* Addressed review comments

* GetLoginPath() updated to return LocalPath

* Updated parameter with LoginTabId instead of TabName

* Fixed an issue where the autocomplete on the country/region field would show the Id instead of the name when using the arrow keys to navigate the autocomplete suggestions

* Fixed whitespace for consistency

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Fixes an issue where folder search would not work in resrouce manager (dnnsoftware#4202)

Closes dnnsoftware#4200

The filtering logic was applied with a null result and removing all the results, I added a check for null which makes the fileting work when the search is used.

* copied from GoogleAnalytics Connector

* added components from DNN.Services.Analytics

* Fixes all warnings in the Console module

- Resolves all stylecop warnings in the Console Admin Module
- Enables TreatWarningsAsErrors on that project

* Removed an unused private variable that was missed

* Update DNN Platform/Admin Modules/Dnn.Modules.Console/Components/BusinessController.cs

* Remove impractical rule

@dnnsoftware/approvers I'm thinking this is a bit overkill, as I can't think of a reason why it would matter who actually clicks the merge button.  The important thing is that there are two approvals.  Do you all agree?

* Resolves 1793 build warnings in Library project (dnnsoftware#4204)

* Resolves 1793 build warnings in DotNetNuke.Collections

This PR covers pretty much all the Collections folder. I am no expert in this area, so any feedback on the documentation is welcome

* Update DNN Platform/Library/Collections/LockingStrategy.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Collections/MonitorLock.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Collections/MonitorLock.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Collections/NaiveLockingList.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Collections/NaiveLockingList.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Collections/SharedList.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/CachedCountryList.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListInfo.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/NavigationManager.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/ServiceScopeContainer.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Collections/ReaderWriterSlimLock.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Collections/SharedDictionary.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Implemented CLSCompliant attribute on DotNetNuke.Abstractions and DotNetNuke.DependencyInjection

* Marked IDNNStartup as not CSL Compliant

* Update DNN Platform/Library/Services/Journal/JournalControllerImpl.cs

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Update DNN Platform/Library/Services/Scheduling/DNNScheduler.cs

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* Update DNN Platform/Library/Services/Search/Internals/LuceneControllerImpl.cs

Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>
Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>

* avoid I/O process frequency during export process.

Fix dnnsoftware#4217

* Ensure comments are only inserted when necessary

Fix dnnsoftware#4219

* Issue-4222 - MembershipModule CurrentCulture fix

* Resolves some build warnings in DDRMenu Project

* Updated .NET Framework requirement in InstallWizard localization files from 4.0 to 4.7.2. also removed mention of older OS versions from the FileAndFolderPermissionCheckFailed.Text key in the InstallWizard localization files

* Moved UserMode from PortalSettings to Personalization class (dnnsoftware#4225)

* Moved UserMode from PortalSettings to Personalization class

Closes dnnsoftware#4213

I am open to any feedback on this, I do believe this way would not be a breaking change...

* Update DNN Platform/Library/Services/Personalization/Personalization.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Services/Personalization/Personalization.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Prevent duplication of code

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Removed finalizer

* Prevent breaking change of Controller public Regex

* Fixed wrong version in deprecation message

* Prevent breaking change on public internal PortalSettings

* Addresses concern with disposing page

* Addresses concern with PortalSettings cast

* Moves things deprecated in v9 to be removed in v11 instead of v10

This moves everything that was marked deprecated in v9.x.x but was slated for removal in v10 to now be removed in v11.

* Corrected one that was done by mistake, it was declared in v6

* Addresses breaking change danger on m_supportsIcons

* Resolves 101 build warnings on CoreMessaging project (dnnsoftware#4216)

* Resolves 101 build warnings on CoreMessaging project

- Resolves 101 warnings
- Prevents sending detailed errors to frontend
- Updates documentation and settings here and there to make it build and correct related documentation
- Enabled TreatBuildWarningsAsErrors on project

* Resolves a build failure due to deprecated PortalID in PortalInfo

* Update DNN Platform/Library/Services/Exceptions/Exceptions.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Resolved all build warnings in RazorHost project (dnnsoftware#4231)

- Resolves all build warnings
- Enables Treat Warnings as Errors on project

* Resolves some build warnings in the ImportExport project. (dnnsoftware#4211)

* Resolves some build warnings in the ImportExport project.

Resolves some (I did not count but probably around 100) build warnings in the Import Export library.

* fix typo

* Brought back 3 removed public methods to prevent creating a breaking change.

* Moved depracation removal to v11

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Resolved all build warnings in Google Analytics Connector (dnnsoftware#4209)

- Resolved all build warnings in Google Analytics Connector
- Enabled TreatWarningsAsErrors on project

* Resolved 249 build warnings in Library project (dnnsoftware#4215)

* Resolved 249 build warnings in Library project

This PR focuses a bit in the Common.Globals file. I have resolved many build warnings and also deprecated some unused public methods that I believe should have never been made public.

All feedback is welcome

* Resolves some more warnings in Globals.cs

* Removed one wrong todo

* Update DNN Platform/DotNetNuke.Abstractions/Application/IApplicationStatusInfo.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Globals.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Addresses PR concerns with v10 vs v11 deprecations

* Update DNN Platform/Library/Common/Globals.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Globals.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Globals.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Removed TODO on IsEditMode

* Removed stale TODO

* Renames some variables for clarity and corrected typos

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Typo fixes

* basically working

* Resolves some more build warnings on library project (dnnsoftware#4236)

* Resolves some more build warnings on library project

I would use some advice on correct replacement in EventManager.cs for the deprecated EventLogController.EventLogType
and
EventLogController.Instance.AddLog call if anyone has some idea on how to refactor that properly. If not, then let's not block this PR for just this and revisit this part later.

* Fix usages of obsolete EventLogController.EventLogType

* Fix test failures

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Added the page ID to the admin ui page management experience

* Removed Module Creator from distribution

- Removes the Module Creator admin module from the distribution as per it's move to https://github.com/DNNCommunity/Dnn.ModuleCreator/releases/tag/v9.8.1
- Also reduces build warnings by about 46

* check the module permission to add module.

Fix dnnsoftware#4234

* Deprecates IEventLogController in favor of DI

Resolves dnnsoftware#4242

The PR:
- Deprecates the interface
- Deprecates the implementation
- The implementation beeing internal and the interface having a warning to not implement by 3rd parties, we can replace it's usage in other downstream classes without too much worry
- I did only one usage replacement in RelationshipController as a proof of concept and adjusted the related test on it.
- There are many many usages of this in Platform, so if thise PR is ok, we have a plan forward for the other usages, any feedback welcome.

* Relaxed compile warnings on Tests projects

Tests projects are not consumed by 3rd parties but where contrained to the same stylecop code style rules as the rest of the platform code (like documentation requirements and name styles and whitespace time. Since we are on a hunt to reduce our build warnings, this PR relaxes those rules on the Tests projects for now.

I would like to revisit this at a later date and actually correct some of those but for now it removes us 959 warnings quickly.

* config sample

* package

* Removed most warnings

* SCAYT language code configuration fixed

* Fix DevOps build badge

It was pointing to an old build definition

* Ensure Localization HTTP Module placement

It needs to be right after URL Rewrite (which determines the culture
from the URL), and then all modules after it can make use of that
culture info

* removed installer for PR

* Applied suggested changes by @valadas

* Fix data reader not getting disposed

Fixes dnnsoftware#3840

* Remove deprecation for some DataProvider methods

Fixes dnnsoftware#3830

* Remove Localization HTTP module from config

It'll get added by 09.08.00.config, having it already in there will
create a duplicate

* Update DNN Platform/Connectors/GoogleTagManager/GoogleTagManager.config

* Run localization after authentication

* Simplify cookie setting to avoid duplicate language cookies being set

* Add skin/page messages after localization has been set

* Added documentation

(cherry picked from commit da5d855)

* Installs JWT provider by default (dnnsoftware#4276)

* Install JWT by default

* Resolves multiple stylecop warnings in JWT project.

* Ensure optional packages that have already been installed are installed so they get upgraded during DNN's upgrade process

* Do not install JWT by default

* do not cache output if page is redirected

* review suggestions incorporated.

* Apply suggestions from code review

* Update DNN Platform/HttpModules/OutputCaching/OutputCacheModule.cs

* Prevents JWT upgrade to change existing config

With the recent change to make optional packages (.resources) upgrade if the package was previously installed, while upgrading the JWT authentication provider, it cause the existing web.config configuration to be reset to the default and that would happen on each upgrade.

This PR makes it ignore the node if it is already present from a previous installation.

* Implement EnablePopups switch in portal settings PB module

* Optimised images with calibre/image-actions

* Fixed an issue where new users would have no preferred locale.

The user preferred locale is a required field in user profiles. Some change made it that in 9.8.0 this property is null. This PR reverts the behavior so the preferred locale is set on new users.

* Fixed an issue where all emails where missing the body

While working on the Feedback module the messages where missing their body, then I tried creating new users and those emails where also missing the body. Looks like the CoreMailProvider was missing setting the body to the emails.

This PR solves that.

* Add Inject module hyperlink and inline editor enabled to personabar site settings module

* fixed build error.

* Resolves some more build warnings on Library project (dnnsoftware#4243)

* Resolves some more build warnings on Library project

* Corrected a typo in IGlobals.cs

* Update DNN Platform/Library/Common/Lists/CachedCountryList.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Internal/TestableGlobals.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListEntryInfoCollection.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListInfo.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListInfo.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListInfo.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListInfo.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Update DNN Platform/Library/Common/Lists/ListEntryInfoCollection.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Use DI for logging

* Update DNN Platform/Library/Common/Lists/ListController.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Fixed tests

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Obsolete the EnableSkinWidgets portal setting and remove it from serialization as this setting is no longer used

* To go with last commit

* Further removal of EnableSkinWidgets

* Tweak to obsolete wording

* Update DNN Platform/DotNetNuke.Abstractions/Portals/IPortalSettings.cs

Co-authored-by: Mitchel Sellers <msellers@Iowacomputergurus.com>

* Update DNN Platform/Library/Entities/Portals/PortalSettings.cs

Co-authored-by: Mitchel Sellers <msellers@Iowacomputergurus.com>

* Rolling back changes

* More stylecop fixes on DotNetNuke.Web project

* More stylcop warnings fixes on library project

Mostly documentaiton and formatting. I recommend reviewer while hiding whitespace only changes 😄

* Apply suggestions from code review

* Fixes invalid extension error message for 0 errors

* Issue-4300 - Password validation lines are fixed

* Fix data provider always failing (dnnsoftware#4302)

* Ensure no expression-bodies properties in DTO objects

* Update bug report template after 9.8.0 release

* remove dragover state from element when dragged BETWEEN pages

* Fix build error

* Fixed AddContentItem to only override the CreatedByUserId if the ContentItem's UserID is >= 0

* Added unit tests

* Fixed last unit test

* Use YAML for build pipeline

* Add triggers config for YAML build

* Fix some integration tests (dnnsoftware#4250)

* Fix test failures in DotNetNuke.Tests.Data

* Fix test failures in Dnn.PersonaBar.Security.Tests

* Fix test failures in ExtensionUrlProviderControllerTests

* Trying to adjust included branches for azure pipeline

* Use latest Cake.Frosting

* Fix some build issues

* Revert to older version of GitVersion

* Show more details from build

* Fix bad build number

* Move Cake logging to later tasks

* Remove tools from NuGet

They're already referenced via UseTool in Program.cs

* Move some logic out of Context class and into task

* Update Cake.NuGet

* Move GetFilesByPatterns into this project

It came from Dnn.CakeUtils, but wasn't compatible with Cake v1

* Install NuGet as a tool

* Fix settings file path

* Trying windows-latest vm image

* Trying to install .net core 2 sdk before build happens

* Trying to preinstall 5.0 SDK

* Added build platform x64

Trying solution found in dotnet/msbuild#5134

* Trying to define the buildPlatform variable differently

* Trying includePreviousVersions

* Use .NET Core 3.1 for build

* Fix build platform value

* Try using .NET 4.8 for Cake.Frosting

* Resolves multple build auto-fixable warnings in solution.

This resolves many many many (6227) build warnings across the whole solution by implementing stylecop auto-fixes on some easy to fix warnings.

I do recommend reviewers to 'hide whitespace changes' while reviewing as this affects many many many files...

The gift that keeps on giving....

* Changed WebConfigurationManager for System.Configuration.ConfigurationManager
when reading connection strings and app settings

* Trying .Net 5.0.100 instead of 5.0.100-rc.2.20479.15

As recommened in cake-build/cake#2957 (reply in thread)

* Removes .Net install from pipeline

* Matching build settings for unit tests

* Should resolve unit tests not running

* Issue-4336 - Umlaut characters in URL cause module setting error (dnnsoftware#4337)

* Issue-4336 - Umlaut characters module settings fix

* Issue-4336 - UTs provided for AdvancedUrlRewriter

* Issue-4336 - Namespace fix

* Issue-4336 - Parallel execution UTs are handled

* Update BUILD documentation for Cake v1 changes

`-Target CreateSettings` and `-Target ResetDevSite` should now be `--target=CreateSettings` and `--target=ResetDevSite`.

* Use variable for cake verbosity

* Update vsdoc file for jQuery to 3.4.1

* Inherit Monaco font (instead of using Roboto)

Fixes dnnsoftware#4344

* Removed elemets and marked obsolete the methods that need to stay due to the public nature of them.

* Adjusted project structure and templates to support proper sitemap configuration on new portal creation. (dnnsoftware#4342)

* Revert "Simplify cookie setting to avoid duplicate language cookies being set"

This reverts commit 2435e14.

* Adjusts build to support upgrades from De-Teleriked Dnn (dnnsoftware#4350)

Allows upgrading from De-Teleriked 9.8.0 instances without reinstalling Telerik dependent packages.

* Don't check user permissions unless there's a user

Fixes dnnsoftware#4355

* Fixed for anti-pattern Obsolete

* Update DNN Platform/Library/Security/PortalSecurity.cs

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Apply suggestions from code review

Code layout fixes

Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>

* Resolves a build error from using deprecated enum value.

Replaced with WebUtility.HtmlEncode

* Fix AssemblyInstaller failing to RemoveBindingRedirect when action="UnRegister" and dll is missing.

* Add assertion to test

* Clean up warnings

Also adjust naming

* DNN-42147 Denying permission (dnnsoftware#4364)

* Adjust flaky test away from ExpectedException

This expresses the intent of the test better (it shows which line it
expects to throw), and locally seems to be more reliable

* Remove duplicate references

Prefer project references to assembly references

* Remove more duplicate references

* Enable warnings as errors for CountryListBox

* Suppress documentation suggestion in test projects

* Turn on TreatWarningsAsErrors for test projects

Co-authored-by: Andrew Hoefling <andrew@hoefling.me>
Co-authored-by: Daniel Valadas <info@danielvaladas.com>
Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>
Co-authored-by: Will Strohl <hismightiness@users.noreply.github.com>
Co-authored-by: David Poindexter <dpoindexter@nvisionative.com>
Co-authored-by: Mitchel Sellers <msellers@Iowacomputergurus.com>
Co-authored-by: Tauqeer Haider <tauqeer.haider@aurea.com>
Co-authored-by: Mandeep Singh <mandeeps@me.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: engineering87 <francesco.delre.87@gmail.com>
Co-authored-by: Muralidaran Kuppusamy <muralidaran.kuppusamy@devfactory.com>
Co-authored-by: Josh Slaughter <josh@clicksandmortarwebsites.com>
Co-authored-by: Stefan Kamphuis <stefan@superska.net>
Co-authored-by: Ben <zyhfish@163.com>
Co-authored-by: rhaiamz <cristian.popescu@avatar-soft.ro>
Co-authored-by: Berk Arslan <berk.arslan@aurea.com>
Co-authored-by: Kieran Corkin <kieran@purplecs.com>
Co-authored-by: Peter Donker <peter@bring2mind.net>
Co-authored-by: David Rodriguez <davidj@intelequia.com>
Co-authored-by: Robert Dima <robert.dima@dnnsharp.com>
Co-authored-by: Mikhail Bigun <mikhail.bigun@ignitetech.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment