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

Fix: Spelling mistake of $date['id']; references to unlock instead of mod; varchar length #59

Merged
merged 3 commits into from
Jun 21, 2023

Conversation

darth3pio
Copy link

  • $date['id'] should be $data['id']
  • Fixed references to unlocks in several comments and strings.
  • varchar length in the database shows that name and longname in game_mod is 24 and 48 respectively, updated validator to reflect this.

…sert

I suspect, unlike the awards.js table where user input prompts for an id, because the id for game mods is an autoincrement value in the game_mod database table the function isn't pulling down the new value upon insertion and complaining about a null value.

This is a possibly temporary fix until I can wrap my head around this specific problem and have ajax handle the table reload as seems to be the way it's handled in other pages within the gamedata module.
@leojonathanoh
Copy link
Member

yay, this looks good now :) Thanks!

Copy link
Member

@leojonathanoh leojonathanoh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@leojonathanoh leojonathanoh merged commit 3d31494 into startersclan:master Jun 21, 2023
6 checks passed
@leojonathanoh leojonathanoh added this to the 3.2.0 milestone Jun 21, 2023
@leojonathanoh
Copy link
Member

By the way, it's difficult for me to find the time to fix some of these things. Really appreciate all your work :)

@darth3pio
Copy link
Author

I figure Docker is a more convenient way to get it all up and running so contributing to this is contributing back into my locally hosted LAN server.

@leojonathanoh
Copy link
Member

it's nice to know there's still a BF2 community, don't know what happened to bf2statistics.com community when I checked last week :(

@darth3pio
Copy link
Author

Yeah, I noticed that, a bunch of Wilson's repos are gone too so I'm assuming he's maybe packed it in? Life stuff perhaps?

Hopefully we'll see him return and if not his legacy will go on.

@darth3pio darth3pio changed the title Fix: Spelling mistake of $date['id']; refrences to unlock instead of mod; varchar length Fix: Spelling mistake of $date['id']; references to unlock instead of mod; varchar length Jun 21, 2023
@leojonathanoh
Copy link
Member

yeah... The BF2Statistics github organization no longer owns the original fork, but the projects are now under ReviveNetwork. Feels like things have moved on. Hopefully there are developers like you that stick around github 😄

@leojonathanoh
Copy link
Member

Just enabled the discussions tab on this repo, perhaps it can fill in something for the BF2Statistics community who use ASP v3 😃

@darth3pio darth3pio deleted the fix-spelling-mistake-xxx branch June 22, 2023 02:00
@darth3pio
Copy link
Author

For sure, I'll be sticking with it, a Git like this is perfect for contributing and sharing code and ideas if nothing else. And with modding still active as shown by all of the recent activity over at ModDB the need to have a backend to host it isn't going away.

My unrealised dream of course, as teased by several comments from Wilson over recent years is how easy it would allegedly be to get Battlefield 2142 support integrated. That's one of my longterm goals.

There's something about that era of Battlefield that has stayed with me.

@leojonathanoh
Copy link
Member

Didn't know BF2142 could be integrated with this stats system? Perhaps the engines are almost the same? I've owned but didn't play 2142 much, so it's pretty much only BF2 for me 😝

@darth3pio
Copy link
Author

darth3pio commented Jun 22, 2023

Wilson bookmarked [imported Google Code repos] a bunch of stuff including (imported Google Code repos), https://github.com/wilson212/bf2142stats_emu, but I assume it was a back of the queue sort of thing. Most of the work would be around the login side of things I suspect. Looking at the game server constants.py the structure looks similar enough that the stats database wouldn't need too much work.

@darth3pio
Copy link
Author

More than anything, I wanna get back into the Star Wars: First Strike mod.

That thing was a heck of a lot of fun.

@leojonathanoh
Copy link
Member

Cool stuff. Good thing the google code repos are automatically imported to github, or a lot of work would have been lost. Perhaps somebody might be bold enough to work on that 😀

Those old games are a heck of fun. My fondest memories are with Counter-Strike, which is still going on strong 😛

@leojonathanoh
Copy link
Member

By the way, from your recent testing, does ASP v3 work properly most of the time? I haven't been using ASP v3 so it'll be nice to know whether it really works, and haven't been following the BF2Statistics community in regard to ASP v3 too, so I don't know about the roadmaps / bugs / new features people were requesting.

At this point, if it's working, we might be able to cut a release for v3.2.0, the changelog's getting a bit too large. 😄

@darth3pio
Copy link
Author

darth3pio commented Jun 23, 2023

Taking a peek at archive.org shows that Wilson was going to release a v4 in January this year but everything went dark, so I'm not sure where he's at. I mostly followed the Github repo as I figured I'd get updates there sooner than I would over at the forums. There's a "ToDo" in the root of ASP but I don't think it's been updated recently so everything that was planned is either on the last post from November or in Wilson's head.

As it currently exists, at least in how it's currently promised, it's mostly complete, beyond the ToDo list I'd be looking at modernising for newer PHP versions and all of the breaking issues with removed PHP functions has been fixed, it generates a few deprecated logs still but that probably won't be an issue till we have to move to PHP 8.0 (as highlighted in the post about v4). Maybe padding out the data.sql with additional entries from contants.py. Beyond that I'd love to expand the player activity system so as it shows when they're online and not just if they've played recently. But we've still get a few more increments to go before we reach our own v4 so that's something that can be added later.

@darth3pio
Copy link
Author

Re-reading that post from November also shows that v4 was going to combine some kind of bulletin board with a bf2sclone replacement so maybe the intention was to replace the forums with v4.

@leojonathanoh
Copy link
Member

Nice information. At least from you described about most fixes already having been done, v3 should work without any major issue. So it should be fine to release v3.2.0 in the next few days.

As for the planned v4, I assume there's no repo as it is, it was really just a roadmap / idea. It'll be kinda nice to see how ASP could fully replace the forums, in that case the soldier's BF2 in-game account would also be their forum account, with bf2sclone stats. The integration would be nice. 😄

@darth3pio
Copy link
Author

@leojonathanoh
Copy link
Member

Hmm seemed like it was being worked on actively, based on ASP v3 system. Hopefully the site comes back up, it was a great community 😢 if not, somebody to keep this going, we will see.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants