Hotfix/fix happiness #905
Merged
Hotfix/fix happiness #905
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Company Agrees. |
| @@ -187,18 +187,26 @@ public void requestMaterialsIfRequired() | |||
| @SuppressWarnings("squid:S135") | |||
| private void requestMaterials() | |||
There was a problem hiding this comment.
The Cyclomatic Complexity of this method "requestMaterials" is 16 which is greater than 15 authorized. 
|
SonarQube analysis reported 46 issues
Watch the comments in this conversation to review them. Top 10 extra issuesNote: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:
|
marchermans
reviewed
Apr 30, 2017
Raycoms
added a commit
that referenced
this pull request
May 4, 2017
* save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting
marvin-bitterlich
pushed a commit
that referenced
this pull request
May 7, 2017
* merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix
marvin-bitterlich
pushed a commit
that referenced
this pull request
May 8, 2017
* merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * fix crashes and exceptions * correct predicate chaining and fix flintNSteel * Fix bug where things don't get dumped correctly * Update InventoryUtils.java * make marvin happy
Raycoms
added a commit
that referenced
this pull request
May 9, 2017
* merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * fix crashes and exceptions * correct predicate chaining and fix flintNSteel * Fix bug where things don't get dumped correctly * Update InventoryUtils.java * make marvin happy * fix citizenData null crash * fix name * improve creative tab * Fix up nullpointer exception * merge builder and lj positioning * merge sonar fix * fix merge bugs * change creativetab
marvin-bitterlich
pushed a commit
that referenced
this pull request
May 11, 2017
* merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * fix crashes and exceptions * correct predicate chaining and fix flintNSteel * Fix bug where things don't get dumped correctly * Update InventoryUtils.java * make marvin happy * fix citizenData null crash * fix name * improve creative tab * Fix up nullpointer exception * merge builder and lj positioning * merge sonar fix * fix merge bugs * change creativetab * fix bug with dman dumping wrong
marvin-bitterlich
added a commit
that referenced
this pull request
May 11, 2017
* merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * fix crashes and exceptions * correct predicate chaining and fix flintNSteel * Fix bug where things don't get dumped correctly * Update InventoryUtils.java * make marvin happy * fix citizenData null crash * fix name * improve creative tab * Fix up nullpointer exception * merge builder and lj positioning * merge sonar fix * fix merge bugs * change creativetab * fix bug with dman dumping wrong
marvin-bitterlich
added a commit
that referenced
this pull request
May 25, 2017
* Create suppressing.MD (#876) * Finishing Base Merge. (#883) * Finishing Base Merge. (#886) * Port/1.11/cap inventory interaction (#818) * Updated the Inventory system to capabilities in 1.11 Signed-off-by: OrionOnline <oriondevelopment@outlook.com> * Add CheckStyle and PMD as plugins to the build.gradle. (#784) * Adding/Updating Style Data. Added: * pmd.xml. Holds the PMD Configuration from the PMD Module of the SonarQube Server. Updated: * checkstyle.xml. Holds the additional style checks SonarQube performs. * Added PMD and Checkstyle gradle plugins. Updated: * build.gradle to add PMD and Checkstyle Gradle-plugins. * CheckStyle.xml to reflect the Minecolonies coding style. * Added PMD and Checkstyle gradle plugins. Updated: * build.gradle to add PMD and Checkstyle Gradle-plugins. * CheckStyle.xml to reflect the Minecolonies coding style. * Some small reformatting to trigger a new build. * Shrink maximal line length to 180 * Switch to TeamCity (#792) * Switch to TeamCity Add support for TeamCity Branch and Build number, having Travis as a fallback * Update build.gradle * trigger a new build * Some last merging fixes - and Formatting of course. Signed-off-by: OrionOnline <oriondevelopment@outlook.com> * Some last merging fixes - and Formatting of course. Signed-off-by: OrionOnline <oriondevelopment@outlook.com> * Small minor changes to make the base change work. Signed-off-by: OrionOnline <oriondevelopment@outlook.com> * Merge Fix/cap based interaction (#800) * Fix the TransferItemsRequestMessage. * Fix NPE when tools are searched in inventories with empty slots. * Fix possible NPE in search for tools. * Some NPE Fixes and Formatting. * Removed getFirstFillableSlotInItemHandler and other related methods. * Changed the way Sonar replacement is handled. It is run once now at the end of a build. * Fixing an Item Duplication bug when items are transferred. Signed-off-by: OrionOnline <oriondevelopment@outlook.com> * Fixing build error that are introduced by merging version/1.11 into the PR branch. Signed-off-by: OrionOnline <oriondevelopment@outlook.com> * Update build.properties * Adding Changelog generation. (#896) Signed-off-by: OrionOnline <oriondevelopment@outlook.com> * Update build.properties (#898) * Port to 1.11 guard builder (#934) * merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * 1.11 - Hotfix for the DMan (#945) * Hotfix for the DMan not behaving properly cause it is not getting reported about the insertion result * Fixing the compile error. * Fix/crashes expceptions (#951) * merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * fix crashes and exceptions * correct predicate chaining and fix flintNSteel * Fix bug where things don't get dumped correctly * Update InventoryUtils.java * make marvin happy * Fix/crashes expceptions (#954) * merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * fix crashes and exceptions * correct predicate chaining and fix flintNSteel * Fix bug where things don't get dumped correctly * Update InventoryUtils.java * make marvin happy * fix citizenData null crash * fix name * improve creative tab * Fix up nullpointer exception * merge builder and lj positioning * merge sonar fix * fix merge bugs * change creativetab * bump version * Bug/1.11 remove some warnings (#965) * Suppress some warning * remove more warnings * Fix warnings for PlacementHandlers * remove more warnings + sonar fixes * fixes sonar * more sonar * Add package info * Fix packages info * Fix/crashes expceptions (#970) (#971) * merge guard improvements * add that but have to remove imports * Fix merge problems. * merge builder changes * Fix merge conflicts * fix some things in placement handler, have to wait for 1.11 capabilities * fix recipe handler * merge saturation * Hotfix/fix happiness (#905) * save happiness and don't devidi through 0 * add constants * check if citizenData != null first * fix assets * another hotfix, new citizen cause crash if happiness 1 * don't double request resources * Fix prboblem with incorrect amount of entities requesting * Fix crash with full inventory on food pickup * fi xnullpointer * delay warehouse more * remove unnecessary import * fix merge issues * optimize imports * merge hotfix * fix crashes and exceptions * correct predicate chaining and fix flintNSteel * Fix bug where things don't get dumped correctly * Update InventoryUtils.java * make marvin happy * fix citizenData null crash * fix name * improve creative tab * Fix up nullpointer exception * merge builder and lj positioning * merge sonar fix * fix merge bugs * change creativetab * fix bug with dman dumping wrong * fix tarzan climbin trees (#975) Uhm I meant the lumberjack * remove not release hut block (#972) * Hotfix/low stats[port to 1.11] (#977) * remove not release hut block * fix bug where citizen start all up with 1 * 1.11 d-man delivery fix (#980) * Fix forceTransferStack return value Fix some java doc * Fix javadoc * Fix happiness calculation about housing * Transfer for real instead of simulating * Fix/builder duping (#984) * remove not release hut block * fix bug where citizen start all up with 1 * Fix problem with builder requesting double * don't request solid placeholder if world is solid * only teleport employed citizen * recalculate after reload * balance happiness depending on guards * Sonar/warnings (#992) * Fix Blockers * first draft for translation constants * fix formatting * fix some constants and other issues * formatting * fix some ternaries * sonar fixes * add suppress * add doc files * reorder * wtf java * last fixes * remove unused * Stop crashing with other world events (#997) * Feature/1.11 better chat (#994) * Suppress some warning * Add worker name to the message when chest is full * remove warnings * Builder say when then start/end a job * Shorter chat version for colony owner * D-man only say that the chest if full when ihe cannot swap D-man say whose chest is full * Change dman saying * Compare itemstack properly * Different message when pick axe requested level is greater than hut level * D-man hold the item he will deliver or the item he did gather Fix javadoc * EntityCitizen does not know about BuildingFarmer now * remove most of the Jobs from EnityCitizen * dman hold the correct item * Feature/builder pickup to 1.11 (#1000) * make builder pickup things extract from lj to interact * sonar fix cast to double * remove unused import * remove double guard code * unused import * Update doc * Refactor even smaller * missing package keyword (#1008) * Refactor/1.11 workorders (#1012) * Suppress some warning * Add worker name to the message when chest is full * remove more warnings * Fix warnings for PlacementHandlers * Builder say when then start a job * WorkOrderBuildHut extends WorkOrderBuildDecoration instead of WorkOrderBuildDecoration extends WorkOrderBuild * revert WorkOrderBuildHut class to WorkOrderBuild * Work Manager does not need to know about Construction tape and WorkOrderBuildDecoration * JobBuilder class does not need to know about WorkOrderBuild * remove duplicated code * fix * Fix javadoc * sonar * Fix sonar issue and suppress sonar warnings * fix sonar issues * sanar fixes * add package info * Sone suppress * bug #1014 fix crash when a colony does not have any owner (#1016) * Fix when colony does not have an owner * remove logging * Bugfix/#758 concurrency issue (#1025) * Centralize the copying of the needed items list, and make it explicitly clear a copy is happening * Remove duplicated methods, and unnecessary casts * Change CopyOnWriteArrayList to ConcurrentLinkedQueue for current warehouse tasks. The list is being modified often, and being treated like a queue. The ConcurrentLinkedQueue should be a better data structure for the situation and be more performant. * Dismount entities before trying to teleport citizens (#1034) * Put item icon in the builder resource tab (#1033) * Check if citizen is jobless before triggering job death achievement (#1037) * Port to 1.11 (waypoints, colony border, sounds, warehouse restriction) (#1032) * merge changes to 1.11 for waypoints * add dman sounds and basics * first merge fixes * also restrict to 1 warehouse per colony * some last merge problems * Fix javadoc error * Add check for invalid line width (#1035) * Feature/1.11 efficient dman (#1017) * gather from the chest which have just been delivered * Deliver mor ethan one stack at a time depending of the deliveryman hut's level * * in BuildingBuilder override neededForWorker instead of using the custom requiresResourceForBuilding method * Add container to building event when the builder did not have to build it * register block in AbstractBuilding instead of adding container only, so that extending class could register any type of block, not only chests * remove checkInWareHouseForFood method in favour of polymorphisme * remove duplicated code * javadoc * language constant * revert back some changes * Used NBT Editor to replace stone brick monster eggs with stone brick (#1041) * Feature/1.11 efficient dman 2 (#1043) * gather from the chest which have just been delivered * Deliver mor ethan one stack at a time depending of the deliveryman hut's level * * in BuildingBuilder override neededForWorker instead of using the custom requiresResourceForBuilding method * Add container to building event when the builder did not have to build it * register block in AbstractBuilding instead of adding container only, so that extending class could register any type of block, not only chests * remove checkInWareHouseForFood method in favour of polymorphisme * remove duplicated code * javadoc * language constant * revert back some changes * Huts need food * annotation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix crash with happiness 1
Changes proposed in this pull request:
Review please