ljacqu
e261650b45
#902 #867 Remove "limboplayer is null" logger statement
2016-08-13 17:33:45 +02:00
ljacqu
eb7487ca84
Revert: Revert "Fix #889 "
...
- Use constructor injection in BungeeService
2016-08-08 21:23:25 +02:00
ljacqu
b4ea396d08
#806 Add tests and avoid sessions cleanup if they're disabled
2016-08-07 10:31:33 +02:00
ljacqu
f2d7fe908e
#806 Create period cleanup task and implement cleanup for session records
2016-08-06 23:20:30 +02:00
ljacqu
eef314b965
Write more unit tests for listeners, minor cleanup
2016-08-06 13:21:39 +02:00
ljacqu
fcb50f2fd5
#887 Migrate int lastlogin column to bigint
...
- Migrate lastlogin column to bigint if it is of int type
- Attempt to migrate timestamp values
- Fix timestamp to bigint migration to create correct timestamps, too
2016-08-05 23:01:58 +02:00
ljacqu
062e461357
#337 Minor improvements in plugin.yml consistency test
...
- Make static field immutable
- Use stricter constraint for checking children paths
- Improve error path logged upon error
2016-08-05 21:27:57 +02:00
ljacqu
69d8232cc8
#337 Check children declarations in consistency test for plugin.yml permissions
2016-08-05 21:23:21 +02:00
ljacqu
f75cd4c5c9
Add unit tests for country validation
2016-08-05 18:57:28 +02:00
ljacqu
766aa154f8
Start unit tests for player movement handler in player listener
2016-08-05 17:08:38 +02:00
ljacqu
d1a1e47928
#872 Harmonize plugin.yml permissions with code + #337 create consistency test
2016-08-04 22:10:31 +02:00
ljacqu
cff6763cee
Add more unit tests for the listeners
2016-08-04 20:48:28 +02:00
ljacqu
8002ef45fd
Teleportation needs to be scheduled in sync
...
- Revert removal of sync scheduled task for teleportation
- Add back listener properties from 5d6f3e1 since they don't interfere
2016-08-04 19:08:05 +02:00
ljacqu
306160e675
Revert changes to PlayerListener
...
- This reverts the changes to PlayerListener.java in 5d6f3e1 to make spawn and first spawn working again
2016-08-04 18:55:56 +02:00
ljacqu
3534f680f7
Fix unit tests
2016-08-04 18:25:43 +02:00
ljacqu
f0dfd85903
#831 Set Spigot profile to be activated by default
...
- I want to be able to check out the source and build immediately
2016-08-04 18:19:38 +02:00
ljacqu
bb52e0120c
Write first unit tests for PlayerListener
2016-08-03 22:11:48 +02:00
ljacqu
4aad1d506c
#831 Implement single session by canceling kicks caused by same name joining
...
- A little less nice, but works on both CraftBukkit and Spigot
- Remove obsolete warning
2016-08-03 21:31:40 +02:00
ljacqu
80337f758b
#831 Cleanup, finalization
...
- Handle AsyncPlayerPreLoginEvent in main PlayerListener class: Spigot emits it in offline mode, whereas offline CraftBukkit doesn't
- Remove listener for sync PlayerPreLoginEvent; not fired by offline CraftBukkit either
- Add warning when offline CraftBukkit is detected and single session setting is enabled
2016-08-03 19:58:10 +02:00
ljacqu
c9718ae86c
Minor - add tests for NewAPI class
2016-07-25 16:39:49 +02:00
ljacqu
19de5a0525
Close #449 Rename NewSetting class to Settings :)
2016-07-23 15:50:40 +02:00
ljacqu
bcc31afb90
#761 Add group options as Property objects and #449 remove legacy Settings class
2016-07-23 15:34:48 +02:00
ljacqu
18a9fbaa26
#449 Migrate remaining non-group legacy Settings
2016-07-23 14:44:52 +02:00
ljacqu
476e0c197d
#873 Save NewAPI object after construction
2016-07-23 10:23:16 +02:00
ljacqu
2d10b46df2
Set ConsoleLogger not to use log file after running ConsoleLoggerTest
2016-07-22 22:49:11 +02:00
ljacqu
cf3d84e3c1
Merge "stringWithLength" matcher implementations
...
- For some reason Hamcrest does not provide a "string with length" matcher -> move our two implementations as one method in AuthMe matchers class
2016-07-22 22:14:10 +02:00
ljacqu
914462fc3f
Unit test ConsoleLogger + clean up and unit test RecoverEmailCommand
2016-07-22 22:09:55 +02:00
ljacqu
a8df8ceb09
#605 Add custom log levels, create debug logging method
...
- Log levels in the log file too
- Create migration from old boolean "stop spam" property to new log level property
2016-07-22 17:45:00 +02:00
ljacqu
8fcb651ebb
Merge branch '784-purgeservice-architecture' of https://github.com/AuthMe-Team/AuthMeReloaded
2016-07-20 21:31:25 +02:00
ljacqu
b671c94e0b
#736 Remove service getters on AuthMe
...
- Remove getX() methods on AuthMe: API classes will be instantiated by injection, too. It doesn't make sense to expose the constructor for the API classes anyway; we are internally responsible for creating these objects and having them created by third-party is not intended
- Remove deprecated annotations on individual methods on API -> whole class is deprecated, annotation on the class is enough
2016-07-20 21:22:09 +02:00
ljacqu
86df740491
#862 Check that LimboPlayer is not null on login (TeleportationService)
2016-07-20 19:10:27 +02:00
ljacqu
3487386840
#835 Fix handling of annotations in DrawDependency tool task
2016-07-19 20:04:55 +02:00
ljacqu
d791fcba94
#835 Remove all injector classes from AuthMe, update installer tasks
2016-07-18 21:47:26 +02:00
ljacqu
57f90fe410
#784 Make DataSource#purgeRecords case-insensitive
2016-07-18 21:29:05 +02:00
ljacqu
cf1032d936
Merge branch 'master' of https://github.com/AuthMe-Team/AuthMeReloaded into 784-purgeservice-architecture
...
Conflicts:
src/test/java/fr/xephi/authme/task/purge/PurgeServiceTest.java
2016-07-18 20:13:43 +02:00
ljacqu
4144281a77
Merge branch '835-integrate-jalu-injector' of https://github.com/AuthMe-Team/AuthMeReloaded
2016-07-18 19:56:12 +02:00
ljacqu
9a534274e5
#835 Fix CommandHandler test class
...
- DelayedInjectionRunner cannot handle mocking the injector as it runs with an injector in the background
2016-07-18 19:50:19 +02:00
ljacqu
3334e2a160
#835 Implement reloading with external injector
2016-07-18 19:39:55 +02:00
ljacqu
9a91156000
#835 Replace injector classes with ones from ch.jalu.injector project
2016-07-17 21:50:48 +02:00
ljacqu
ca4a64f398
#844 Run PurgeTask as repeating task and #784 write tests for PurgeTask
2016-07-17 21:12:01 +02:00
ljacqu
5953bfd012
#784 Separate purge execution and purge task creation
...
- Delete accounts in PurgeExecutor, after permission check
2016-07-17 17:33:25 +02:00
ljacqu
2a4cda0709
#784 Perform bypass purge permission check with OfflinePlayer objects
...
- Move permission check inside PurgeTask to perform it with OfflinePlayer objects instead of lowercase names
- Move purge members into child "purge" package
- Unify online and offline default permission behavior in DefaultPermission
2016-07-17 11:54:22 +02:00
ljacqu
674a051586
#762 Clear all stored sessions if feature gets disabled on reload
2016-07-15 19:47:42 +02:00
ljacqu
3b06524796
#762 Use timestamps for session + #816 Logout should invalidate a player's session
2016-07-15 19:35:35 +02:00
ljacqu
10392d0d65
#856 Handle null Location on PlayerCache object
...
- Location may null when read from file
2016-07-15 18:49:01 +02:00
ljacqu
160cbc6aa4
#849 Catch exception in TwoFactor hash and write unit tests
2016-07-15 18:23:18 +02:00
ljacqu
42c3dcbbcb
Revert code changes to single session
2016-07-12 22:23:53 +02:00
ljacqu
4a41f51384
Merge remote-tracking branch 'origin/master'
2016-07-12 22:15:28 +02:00
ljacqu
8ce1d53403
Merge remote-tracking branch 'origin/master'
2016-07-12 22:15:22 +02:00
ljacqu
e7b980d435
#605 Logger - name methods after their log level
...
- Remove separate print stacktrace method
- Log level into the log similar to console output
2016-07-12 22:06:32 +02:00
ljacqu
dccbd5262f
Don't update IP twice
...
- IP is updated in datasource#updateSession after successful login
- Remove no longer used DataSource#updateIp
2016-07-12 21:45:05 +02:00
ljacqu
8d54557f3d
Add tests for PlayerDataStorage and LimboCache
2016-07-11 21:09:15 +02:00
ljacqu
97274d8c19
#843 Fix NPE upon registration
...
- Let login classes handle the login process, remove duplications in register classes
2016-07-11 20:42:28 +02:00
ljacqu
a6d885e0bb
Write tests for PlayerDataStorage
2016-07-10 23:00:58 +02:00
ljacqu
8e91636fbd
Create tests for LimboCache
2016-07-10 17:15:27 +02:00
ljacqu
c9c4e69e1b
#838 Fix force spawn on join setting
...
- Add call to teleport service after login and logout
2016-07-10 13:04:35 +02:00
ljacqu
491853e0c8
Change github URLs from AuthMe-Team/ to AuthMe/
...
- Change README sections to use more markdown
2016-07-10 11:42:40 +02:00
ljacqu
faddb3ffac
No longer need to call setup() directly in ProtocolLibService constructor
2016-07-09 09:25:36 +02:00
ljacqu
aa6fe3e874
Rename method on SettingsDependent
...
- Rename loadSettings to reload
- Make ProtocolLibService only implement SettingsDependent, not Relaodable alongside
2016-07-09 09:24:25 +02:00
ljacqu
42ef99cf15
Update verification comments in message files
2016-07-06 21:41:43 +02:00
ljacqu
78aac256fd
#834 Add Polish messages by @RikoDEV
2016-07-06 21:40:26 +02:00
ljacqu
693a4500ef
Fix problem after merge
2016-07-06 21:13:32 +02:00
ljacqu
d297d0b528
Merge branch 'jsoncache-fix' of https://github.com/AuthMe-Team/AuthMeReloaded
2016-07-06 20:55:56 +02:00
ljacqu
d8f6fb085e
Cleanup - remove unused elements in code
2016-07-04 21:53:22 +02:00
ljacqu
ea6603a6dc
Fix registered players being teleported to first spawn, fix player losing his speed
2016-07-04 21:38:23 +02:00
ljacqu
2867ebaddd
Fix failing test
2016-07-04 20:31:40 +02:00
ljacqu
1c3ca520dc
Fix player being unable to walk after admin registration
...
- Fix sessions still working after being disabled on reload
2016-07-04 19:56:44 +02:00
ljacqu
c040623739
Merge 765-teleport-handling into jsoncache-fix
2016-07-04 18:47:34 +02:00
ljacqu
06322f37bf
#765 Don't teleport after registration; make UnregisterAdminCommand use the unregister process
2016-07-03 21:57:44 +02:00
ljacqu
7788ad6230
#765 Replace Utils usages with TeleportService + misc householding
...
- Remove some legacy settings
- Move Utils "addNormal" method to AuthGroupHandler
(Reapplied changes from orphaned fe29089)
2016-07-03 19:55:31 +02:00
ljacqu
8f5817883e
Testing - check return value of Set#add instead of separately checking Set#contains
2016-07-03 11:47:37 +02:00
ljacqu
2420a83ec3
#797 Add missing French messages
...
- Thanks to @Twonox
2016-07-03 09:49:38 +02:00
ljacqu
a9574d2363
#819 Email text migration - migrate %tag% style placeholders as well
2016-07-02 22:22:03 +02:00
ljacqu
3984208548
#797 Make two messages translatable
2016-07-02 21:52:50 +02:00
ljacqu
9a5c432509
#655 Encapsulate mail service
...
- Change SendMailSSL to be injected into classes and created regardless of settings
- Various minor cleanups (remove accidentally committed test, add more precise logging statement)
2016-07-02 20:56:53 +02:00
ljacqu
d2556b8a04
#817 Sessions timeout - remove mention of 0 = unlimited in config comments
2016-07-02 19:39:13 +02:00
ljacqu
22911a0bb9
Remove duplicate column initialization code, add datasource integration tests
...
- Drop initialization of all columns on table create in favor of checking each column individually. This is slower but guarantees that each column is only defined once in the code. Columns are only created once so having clean code outweighs performance.
- Write more datasource integration tests
2016-07-02 19:16:26 +02:00
ljacqu
f1d5f3df28
Merge conversion classes to one class per conversion type
...
- Fix issues in #736 using service getters
2016-07-02 15:18:13 +02:00
ljacqu
0283395da4
Merge branch 'master' of https://github.com/AuthMe-Team/AuthMeReloaded into jsoncache-fix
...
Conflicts:
src/main/java/fr/xephi/authme/listener/protocollib/ProtocolLibService.java
2016-07-02 11:01:46 +02:00
ljacqu
70226f7ddb
#813 Move test resources to conform to package structure
2016-07-02 10:57:30 +02:00
ljacqu
de3f3a42ab
#679 Fix compatibility with InnoDB/XtraDB engine
...
- Includes other improvements in collation and data types
2016-07-02 10:24:44 +02:00
ljacqu
1f35ccd841
Improve bat helpers / minor changes to logging and todo messages
2016-07-02 10:17:52 +02:00
ljacqu
73a9b5ce0c
Cleanup - reduce duplication in MySQL class
2016-07-02 08:28:37 +02:00
ljacqu
6585b68749
Fix TeleportationService tests + rename methods
...
- Fix and supplement unit tests for TeleportationService
- Rename methods as to avoid confusion (login vs. LoginEvent when player joins)
- Add javadoc with note about Player#hasPlayedBefore always being false
2016-06-30 22:38:36 +02:00
ljacqu
1cadbd2345
Minor code householding
...
Various neglectable changes while looking at recent commits
- Remove Inject annotation on ProtocolLib adapters that are no longer created via injection
- Reduce ProtocolLib adapters to package private - interaction always goes via ProtocolLibService
- Fix typos
2016-06-30 20:50:13 +02:00
ljacqu
1361174892
Encapsulate GeoLiteAPI class
2016-06-27 21:32:52 +02:00
ljacqu
d72d6ddf5a
AntiBot - make public field private
2016-06-26 09:25:52 +02:00
ljacqu
df060ff29c
Remove legacy setting + remove obsolete todo comment (wontfix #492 )
...
- Remove migrated legacy setting
- Remove months old todo comment
- Remove large chunk of commented out code and merge two explanations
2016-06-25 01:14:01 +02:00
ljacqu
6812cfa4db
Add utility for safe pattern compiling
2016-06-24 23:50:11 +02:00
ljacqu
54ababdd28
#788 remove duplicate FR message + #772 update missing message comments
2016-06-24 18:59:25 +02:00
ljacqu
e7ba579960
#778 Delayed runner: add support for annotations, add validation
...
- Add support for dependencies identified by annotations
- Add some more usage validation
- Change a few test classes to use the DelayedInjectionRunner
2016-06-19 22:54:12 +02:00
ljacqu
4b3ab4b116
#769 Create integration test for service initialization in onEnable()
...
- Test that services can be instantiated (e.g. no circular dependencies)
- Instantiate Messages via injection instead of manually
2016-06-19 21:16:46 +02:00
ljacqu
1377b10c24
Simplify instantiation of permission handlers
...
- Change permission handler to retrieve the services it needs, instead of getting them from the PermissionsManager
- Reduce visibility of some methods
- Add Reloadable interface
2016-06-19 18:20:12 +02:00
ljacqu
ff2f8a63bb
Fix bugs in purge classes, cleanup
...
- OfflinePlayer objects were never mapped to names
- Essentials purge was run even if setting was disabled
- Console user saw messages twice
- Misc: trivial field reorderings, change DefaultPermission to be null safe
- Permissions: do not expose PermissionHandler just to get its type
2016-06-19 17:56:50 +02:00
ljacqu
a0abe3d19d
Minor householding
...
- Remove getter for NewSetting
- Reduce visibility of classes used by DelayedRunner
2016-06-19 16:13:00 +02:00
ljacqu
d35005167e
Merge branch 'master' of https://github.com/AuthMe-Team/AuthMeReloaded into 674-purge-process-refactor
2016-06-19 16:02:08 +02:00
ljacqu
e38c1ea447
Merge branch '777-Fast-Join-Handler' of https://github.com/AuthMe-Team/AuthMeReloaded
2016-06-19 15:51:40 +02:00
ljacqu
fb5e7d40c6
Change delayed injection to only change behavior of its own elements
...
- Introduce new BeforeInjecting annotation to not modify the (expected) order of annotations that do not belong to us
- Avoids using hacky way of first initializing field to an Answer that will delegate on demand to the proper class...
- Remove PostConstruct support for Mockito's InjectMocks: we should not change the established behavior of external elements
2016-06-18 15:31:11 +02:00
ljacqu
a1c62e7c04
Create delayed injection test runner
...
- Test runner supporting new "DelayedInjection" annotation: such fields are only initialized with instantiation right before the first time they're used in tests, allowing to set up mock behavior beforehand
2016-06-18 13:19:07 +02:00
ljacqu
cd1acfde1b
#674 Create tests for purge commands and purge service
2016-06-18 11:13:17 +02:00