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

Added SetDoDaylightCycle() and IsDaylightCycleEnabled() to cWorld. #1296

Merged
merged 7 commits into from Aug 12, 2014

Conversation

Projects
None yet
5 participants
@Howaner
Contributor

Howaner commented Aug 6, 2014

I need this for a GameRule plugin.

@archshift

This comment has been minimized.

Show comment
Hide comment
@archshift

archshift Aug 6, 2014

Contributor

I think CycleDaylight would be a better name for the bool than DoDaylightCycle, but there's probably a better name that I can't think of right now.

Contributor

archshift commented Aug 6, 2014

I think CycleDaylight would be a better name for the bool than DoDaylightCycle, but there's probably a better name that I can't think of right now.

@Howaner

This comment has been minimized.

Show comment
Hide comment
@Howaner

Howaner Aug 7, 2014

Contributor

And what is a good name for the functions?
SetCycleDaylight(); GetCycleDaylight() ?

Contributor

Howaner commented Aug 7, 2014

And what is a good name for the functions?
SetCycleDaylight(); GetCycleDaylight() ?

Show outdated Hide outdated src/World.h
@tigerw

This comment has been minimized.

Show comment
Hide comment
@tigerw

tigerw Aug 8, 2014

Member

Ahem.

Member

tigerw commented Aug 8, 2014

Ahem.

@tigerw

This comment has been minimized.

Show comment
Hide comment
@tigerw

tigerw Aug 8, 2014

Member

Don't forget to write m_IsDaylightCycleEnabled to file in cWorld's destructor, as it is plugin changeable.

Member

tigerw commented Aug 8, 2014

Don't forget to write m_IsDaylightCycleEnabled to file in cWorld's destructor, as it is plugin changeable.

@Howaner

This comment has been minimized.

Show comment
Hide comment
@Howaner

Howaner Aug 10, 2014

Contributor

done.

Contributor

Howaner commented Aug 10, 2014

done.

Show outdated Hide outdated src/ClientHandle.cpp
@Howaner

This comment has been minimized.

Show comment
Hide comment
@Howaner

Howaner Aug 10, 2014

Contributor

done.

Contributor

Howaner commented Aug 10, 2014

done.

Show outdated Hide outdated src/ClientHandle.cpp
Show outdated Hide outdated src/ClientHandle.cpp
@Howaner

This comment has been minimized.

Show comment
Hide comment
@Howaner

Howaner Aug 10, 2014

Contributor

done.

Contributor

Howaner commented Aug 10, 2014

done.

@madmaxoft

This comment has been minimized.

Show comment
Hide comment
@madmaxoft

madmaxoft Aug 11, 2014

Member

Me likes. If there are no other objections, merge.

Member

madmaxoft commented Aug 11, 2014

Me likes. If there are no other objections, merge.

Howaner added a commit that referenced this pull request Aug 12, 2014

Merge pull request #1296 from mc-server/LuaAPI
Added SetDoDaylightCycle() and IsDaylightCycleEnabled() to cWorld.

@Howaner Howaner merged commit 8f0c491 into master Aug 12, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details

@archshift archshift deleted the LuaAPI branch Aug 12, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment