Add Scheduled Matches to mv_teams

mfranchetti commented 8 years ago

The mv_teams table has a played field to record the number of matches played. I am proposing that we add a scheduled field to record the number of matches that the team has scheduled but unplayed. This will require updating the function syncMVteam in order to correctly populate the field. The benefit is that we can then include teams who have completed matches or scheduled matches when syncMVteam is called. League tables will then show all teams included in a scheduled league from the start rather than only after their first match.

Issues

How are table updates being included? Do i just create a new version update or is there one that should be used.
Are there any other functions that would be affected by the new field? I don't believe so, as I think syncMVteam is the only one that updates mv_teams and any that read from it should be specifying fields.
In order for this to update correctly, 'syncMVteam' will need to be called after scheduled matches are added. I doubt that is the case now but have not confirmed. Should be a simple job tho'.

I'm happy to handle this, but wanted to ensure it was acceptable and not conflicting with any other developments.

mfranchetti commented 8 years ago

I've managed to implement this on my own OBBLM install (www.wbbl.co.uk). It required:

minor change to call_match.php
new field in table mv_teams
change to MySQL procedure syncAllMVs
change to MySQL function syncMVteams

Are we using MySQL_upgrade_queries 096-097?

thefloppy1 commented 8 years ago

I would like to see that upgrade querie chaned to 097-101 or some. Also I would like to test this out can you send me the files.

mfranchetti commented 8 years ago

Sure, I will try and do a commit this evening (I've been away for the weekend) or I can e-mail the files directly if that's easier to test?

thefloppy1 commented 8 years ago

Email directly is easy On 4 Jul 2016 6:48 PM, "Michael" notifications@github.com wrote:

Sure, I will try and do a commit this evening (I've been away for the weekend) or I can e-mail the files directly if that's easier to test?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/TheNAF/naflm/issues/139#issuecomment-230238478, or mute the thread https://github.com/notifications/unsubscribe/APXWho_5DU9OfsLfhbHY-lLQjWEb_29Bks5qSMjOgaJpZM4JBBbZ .

mfranchetti commented 8 years ago

I'll need an email address for you then :)

rythos42 commented 8 years ago

Definitely make it 97-101. My hope for 101 is to upgrade the upgrade process, but it will almost certainly use the use existing infrastructure.

TheNAF / naflm

Add Scheduled Matches to mv_teams #139