Rewrite setup.pl

[ehuelsmann]

This issue aims to achieve multiple goals:

1. Separate logins to the main application (login.pl) from setup application
2. Improve the setup user interface (and user experience)
3. Standardize route declaration
4. Simplify URL parsing and route dispatch
5. Outsource web-framework maintenance (hook into community efforts)
6. Simplify the wrapper code for the '/setup(.pl)' routes
7. Increase testability of the code through unit tests (in addition to web tests)

These goals are achieved by:

1. Using Dancer as the web framework (addresses (3), (4) and (5))
2. Configuring Dancer to use cookie auth with separate cookies for /setup and /login (addresses (1)
3. Refactoring application functions - beyond wrapping/mapping to libraries - out of /setup into libraries (addresses (6) and (7))
4. Using a standard web-framework which allows a broader range contributors to contribute to UI improvements

These changes require the following steps to be executed:

• Establish a Dancer integration/ baseline for /setup
  (includes Dancer auth plugins integrating with LedgerSMB type authentication)
• Rewrite the functons of /setup
  • System information / configuration overview
  • Listing available databases
  • Creation of new databases
  • Database sanity checks
  • Copying of databases
  • Creating database backups
  • Creating database role backups
  • Listing of users
  • Creation of new users
  • Modification of user role assignments
  • User password resets
  • Loading of template sets
  • Schema upgrades
    • From LedgerSMB 1.2/1.3
    • From SQL Ledger 2.8/3.0

From #4408, take into account this request:

Suggestion: Adding a way to add/create a database once you have logged in as admin on the setup.pl because if you do not provide a database name and create at the login section of the setup.pl, a list of database is presented to you as a DBA, but you do not have a label + textbox + button to create a database from that exact location.

A simple GET redirection would probably do, since the user already logged ?

Thank you !

From #3314, please consider:

More small issues with setup.pl:

I have the keys, but cannot logout to lock up when I leave.

The field hints for importing vs. creating a user could say something about Postgres databases vs. other lsmb databases. Or the program could catch, recover, query and move forward.

User list is slammed against the left jamb.

On all or most setup.pl pages, there is space to have help paragraphs to the right, like many SOHO routers and network interfaces.

Conversely, space could used to combine the user list and user entry pages to make entry smoother. But
just returning to the user list from user entry should suffice.

[gwhitney]

In the attempted installation from github #4655, I find that I am unable to reset a user password from /setup.pl. I see that the "user password resets" box above is unchecked. Does this mean that it is not currently expected to be able to reset a user password from setup.pl? I wanted to check before I filed a new issue for something that is already known. Thanks for clarifying.

[ehuelsmann]

From #1198 (contributed by @sbts):

There are a number of user experience issues with the current implementation of setup.pl

The first one is we need some help information on the login screen. at the moment it is expected that users just know that the Database field will either

• create a new db if the text is not a valid db name
• allow changes to and/or backup of an existing db if the text is a valid db name

Really the Database field is probably not needed here these days.
Instead

• simple login screen
• once logged in present a list or dropdown list of available DB's
• make the list filterable (dojo can filter select elements)
• add a button "Create New Company Database"
• add text that makes it clear that the company name is the database name

Once you have opened a config page for a specific DB there are a couple of other issues.

• If the db is not a recognised ledgersmb db the cancel button should not be labeled "Yes"
  but either "cancel" or "back".
• The backup buttons should not be active, or not visible at all.
• when the cancel or back button is clicked it should return to the list, rather than the login screen

Which got this response by @ehuelsmann :

First we need to define what setup.pl means to be/contribute

[ehuelsmann]

Given that we're moving to Vue and web services and the fact that there is no real good services infrastructure in Dancer, this looks like an idea more than an actionable enhancement. Converting to an idea discussion type.

[neilt]

Attached is a proposed prototype layout.

t9.html.zip

[howardlowndes]

Black on blue title bar is a bad start http://webaim.org
Can the base level menu be presented from the start instead of only the title bar?
Can the back button be used to return to the base level menu instead of having to climb back thru the previous screens (I am viewing this in Chrome on a tablet which may make a difference).
My two bob's worth...

[neilt]

Agree to 1, no attempt was made to style this prototype. I did try to make the blue lighter, but could not find the correct magic. Quasar talks about light-blue-2 at https://quasar.dev/style/color-palette/, but that does not work.

Agree to 2 also. Just did not get around it it.

On 3, what is wrong with just clicking on a menu item to go back?

[howardlowndes]

Re 3. I think (I can't find the screen at the moment), I think that on my tablet, when a screen is open, the side menu bar is missing. I'm not sure, I'll need to check again tomorrow.

…

On Wed, 3 Jan 2024, 08:51 Neil Tiffin, ***@***.***> wrote: Agree to 1, no attempt was made to style this prototype. I did try to make the blue lighter, but could not find the correct magic. Quasar talks about light-blue-2 at https://quasar.dev/style/color-palette/, but that does not work. Agree to 2 also. Just did not get around it it. On #3 <#3>, what is wrong with just clicking on a menu item to go back? — Reply to this email directly, view it on GitHub <#7094 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AMS2F7TSPT4DB6EZILDOM6DYMR6PNAVCNFSM6AAAAABBKRUMFWVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM3TSOJXGIZDE> . You are receiving this because you commented.Message ID: ***@***.***>

[neilt]

Try tapping on the 3 horizontal bars as shown in the attached image.

[neilt]

Now that I think about it, Quasar's design is responsive. That means that the widgets have to be self contained. This is fundamentally different that the current fixed (desktop) design used in LSMB. Part of the challenge is going to be selecting fixed, responsive, or adaptive web design as the way forward for LSMB? If not fixed or adaptive, then that will require not only new widgets, but a mindset change to the UI. Adaptive will be more complex. Responsive will have more limited UI components.

[howardlowndes]

I do recall that when the screen first appears it is the full screen that appears and the text on the title bar is so small, and being black on dark blue, insignificant that I had to expand the screen to read it. That may have pushed the three bars off the screen to the top left when trying to study the screen itself making them forgettable, hence the use of the back button.

…

On Wed, 3 Jan 2024, 22:04 Howard Lowndes, ***@***.***> wrote: Re 3. I think (I can't find the screen at the moment), I think that on my tablet, when a screen is open, the side menu bar is missing. I'm not sure, I'll need to check again tomorrow. On Wed, 3 Jan 2024, 08:51 Neil Tiffin, ***@***.***> wrote: > Agree to 1, no attempt was made to style this prototype. I did try to > make the blue lighter, but could not find the correct magic. Quasar talks > about light-blue-2 at https://quasar.dev/style/color-palette/, but that > does not work. > > Agree to 2 also. Just did not get around it it. > > On #3 <#3>, what is wrong > with just clicking on a menu item to go back? > > — > Reply to this email directly, view it on GitHub > <#7094 (reply in thread)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AMS2F7TSPT4DB6EZILDOM6DYMR6PNAVCNFSM6AAAAABBKRUMFWVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM3TSOJXGIZDE> > . > You are receiving this because you commented.Message ID: > ***@***.***> >

[neilt]

@ylavoie Found a way to wrap the q-input so the label is on the left.

See https://github.com/ylavoie/LedgerSMB/blob/quasar1/UI/src/views/LoginPage.vue