mahtab2003 / Xera

Now Introducing a new and easy way to manage your clients and MyOwnFreeHost hosting accounts.
https://xera.eu.org/
GNU General Public License v2.0
47 stars 39 forks source link

SitePro Fix #53

Closed gilect-com closed 2 years ago

gilect-com commented 2 years ago

Fixed SitePro from creating a website in the wrong directory, when using a domain other than the Account Domain. Previously it was created at /htdocs/domain When it should be at /domain/htdocs instead

greenreader9 commented 2 years ago

DO NOT DO THIS!!!

This is will break the base domain of the account (The domain that the account was created with).

The solution is to programmatically determine what to do, there is not a "Always This" or "Always That" solution.

gilect-com commented 2 years ago

Why exactly is this breaking the original domain?

All this is doing is preventing files from being created in /htdocs/domain (a directory which does not exist) Instead they should be created in /domain/htdocs/

Using the website builder on the original Account Domain still works fine, it creates the files in /htdocs/

4yx commented 2 years ago

This will break original domain on the account

Could you explain how it would break it?

greenreader9 commented 2 years ago

Ah, I think I misread the code, stupid GitHub app did not show me what I needed to see.

My apologies, this should actually work.

@mahtab2003 Can you merge this? Thanks

4yx commented 2 years ago

It looks good from my end as well, I'll attempt to test it out.

gilect-com commented 2 years ago

It should work, all it does it changes the directory where SitePro files are created. I applied the change myself and it fixes the SitePro bug that previously occurred.

4yx commented 2 years ago

This should be safe to merge, yes. @mahtab2003 could you reopen and merge this?

sonarcloud[bot] commented 2 years ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication