Sigrotor + Coding Help + Mini Project

Deleted member 77746 · 4 Feb 2021 at 21:21

visibleman said:
Done (on fork) - it's scruffy but works.

Just added the code to live site and it doesn't appear to copy the index.php file. :confused:

Deleted member 77746 · 4 Feb 2021 at 21:47

visibleman said:
Currently, it'll only copy across when the /signatures/{username}/ directory is being created. But i've pushed another version that will continuously check (on upload) to see if the script exists within the users directory and copy across accordingly.

I don't see any code changes in your fork?

Deleted member 77746 · 4 Feb 2021 at 22:13

touch said:
Wouldn't it be easier to have that file once in the signature root directory and have the urls given to the users a path to that file with their username as a parameter, then you can search for files in the user's folder using the passed username parameter. Then if you need to change the file you don't need to go through every directory and update them all.

hey touch, I noticed you forked the site too. If you think you can assist feel free to edit the code on your fork and I'll check it out see.

I have no idea what will be easier or not. I'm just going with the flow till something works.

Deleted member 77746 · 4 Feb 2021 at 22:21

touch said:
Sorry, just delete anything I've added. Didn't think I've committed anything. I've never used git before so I was just trying it out to see how it all works. I only use VisualStudio so I could get it all connected up and download the files but it doesn't run PHP so I couldn't test anything and didn't want to break stuff.

You won't break it because if you fork on Github the code is edited on a "copy" then I would need to pull/merge the requested code change.

Don't worry you haven't broken anything.

No seriously if you think of a better way please do commit. I'm all up for helpers as I don't code myself I just mess about with things till it works.

Deleted member 77746 · 5 Feb 2021 at 10:38

Spot on, changes made to live site, lets see if this works.

Deleted member 77746 · 5 Feb 2021 at 11:04

@visibleman

Did the code change but get the following error message all the time on image upload. Trying to check the code now see what's wrong.

"Error with setup, contact admin!"

Deleted member 77746 · 5 Feb 2021 at 14:20

visibleman said:
The index file under /signatures/ needs to exist otherwise it'll throw that nondescript error message as there's nothing to copy.
Also make sure all the defines are correct if you're manually copy the changes.

I will do a rewrite rule variant when I get half an hour, as the current solution is pretty horrible.

ah yes it was down to that file not existing it works! just tested it.

It's a working site now so can be used!

https://www.overclockers.co.uk/forums/threads/sigrotor-is-back-2021.18919999/