[Pharo-users] [Pharo 70] Report period: 9 December 2017 to 20 January 2018

Thank you all for all these improvements... * 20953-Gitlab Private Project Support (via Iceberg) >> Includes pharo-vcs/iceberg#558: Gitlab Private Project Support Issue URL: https://pharo.fogbugz.com/f/cases/20953 PR URL: https://github.com/pharo-project/pharo/pull/667 Diff URL: https://github.com/pharo-project/pharo/pull/667/files Thanks to seandenigris * 20992-Use-Stdio-class-instead-of-VTermOutputDriver >> Remove dependency to VTermOutputDriver in InspectorNavigator >> Fix consistency pb in Stdio class Issue URL: https://pharo.fogbugz.com/f/cases/20992 PR URL: https://github.com/pharo-project/pharo/pull/691 Diff URL: https://github.com/pharo-project/pharo/pull/691/files Thanks to VincentBlondeau * 20942-Integrate-Epicea-826-in-Ph7 >> Upgrade Epicea to 8.2.7 (but keep Cyril's contribution in this repository which is not copied upstream yet). >> >> Changes: >> >> - Fix case 20223. >> - Fix EpLog and OmFileStore #entryReferences, the result must have the same order as #entries. >> - Improve apply preview of class comments. >> - Sync with changes in Pharo 7: remove override of ZnBufferedWriteStream>>#cr since it''s already present in latest version of Zinc. >> - Improve some code comments. >> - Fix refactoring apply (general case) >> - Fix class rename refactoring Issue URL: https://pharo.fogbugz.com/f/cases/20942 PR URL: https://github.com/pharo-project/pharo/pull/690 Diff URL: https://github.com/pharo-project/pharo/pull/690/files Thanks to tinchodias * 20972-ShiftClassInstaller-do-not-announce-ClassModificationApplied-event >> Class changes should notify ClassModificationApplied for Pharo6 compatibility >> >> https://pharo.fogbugz.com/f/cases/20972/ShiftClassInstaller-do-not-announce-... Issue URL: https://pharo.fogbugz.com/f/cases/20972 PR URL: https://github.com/pharo-project/pharo/pull/684 Diff URL: https://github.com/pharo-project/pharo/pull/684/files Thanks to dionisiydk * 20962-Rubric-should-not-depends-on-NautilusCommons2 >> RubCommentAnnotationDisplayer is moved to Nautilus under tag Rubric >> >> https://pharo.fogbugz.com/f/cases/20962/Rubric-should-not-depends-on-Nautilu... Issue URL: https://pharo.fogbugz.com/f/cases/20962 PR URL: https://github.com/pharo-project/pharo/pull/688 Diff URL: https://github.com/pharo-project/pharo/pull/688/files Thanks to dionisiydk * 20979-Introducing-DoubleWord-and-DoubleByte-arrays >> https://pharo.fogbugz.com/f/cases/20979/Introducing-DoubleWord-and-DoubleByt... >> >> (Needed for VMMaker compatibility) Issue URL: https://pharo.fogbugz.com/f/cases/20979 PR URL: https://github.com/pharo-project/pharo/pull/689 Diff URL: https://github.com/pharo-project/pharo/pull/689/files Thanks to clementbera * 20978-PharoIssueTest-should-be-long-time-test >> 1 minute time limit for PharoIssueTest. >> >> https://pharo.fogbugz.com/f/cases/20978/PharoIssueTest-should-be-long-time-t... Issue URL: https://pharo.fogbugz.com/f/cases/20978 PR URL: https://github.com/pharo-project/pharo/pull/687 Diff URL: https://github.com/pharo-project/pharo/pull/687/files Thanks to dionisiydk * 20936-64bit-primitiveFailure-for-10-largeIdentityHash >> https://pharo.fogbugz.com/f/cases/20936/64bit-primitiveFailure-for-1-0-large... Issue URL: https://pharo.fogbugz.com/f/cases/20936 PR URL: https://github.com/pharo-project/pharo/pull/653 Diff URL: https://github.com/pharo-project/pharo/pull/653/files Thanks to Ducasse * 20976-wrong-FLPharo7Platform-initialization >> https://pharo.fogbugz.com/f/cases/20976/wrong-FLPharo7Platform-initializatio... Issue URL: https://pharo.fogbugz.com/f/cases/20976 PR URL: https://github.com/pharo-project/pharo/pull/686 Diff URL: https://github.com/pharo-project/pharo/pull/686/files Thanks to pavel-krivanek * 20975-Fix-baselines-after-Fuel-update-case-20970 >> https://pharo.fogbugz.com/f/cases/20975/Fix-baselines-after-Fuel-update-case... Issue URL: https://pharo.fogbugz.com/f/cases/20975 PR URL: https://github.com/pharo-project/pharo/pull/685 Diff URL: https://github.com/pharo-project/pharo/pull/685/files Thanks to pavel-krivanek * 20970 remove pharo 6 platform class from pharo 7 >> Added Fuel platform class for Pharo 7. >> Removed Fuel platform class for Pharo 6. >> >> Replaces https://github.com/pharo-project/pharo/pull/674. >> >> https://pharo.fogbugz.com/f/cases/20970/Remove-Pharo-6-platform-class-from-P... >> >> https://github.com/theseion/Fuel/issues/227 Issue URL: https://pharo.fogbugz.com/f/cases/20970 remove pharo 6 platform class from pharo 7 PR URL: https://github.com/pharo-project/pharo/pull/682 Diff URL: https://github.com/pharo-project/pharo/pull/682/files Thanks to theseion * 20952 Super setUp need to be called in ExampleSetTest, SessionManagerRegistrationOrderTest, GTS and GLM prefixed tests >> add missing super setUp calls. >> >> https://pharo.fogbugz.com/f/cases/20952/Super-setUp-need-to-be-called-in-Exa... Issue URL: https://pharo.fogbugz.com/f/cases/20952 Super setUp need to be called in ExampleSetTest, SessionManagerRegistrationOrderTest, GTS and GLM prefixed tests PR URL: https://github.com/pharo-project/pharo/pull/666 Diff URL: https://github.com/pharo-project/pharo/pull/666/files Thanks to astares * 20968 Unused temps in various GML methods >> >> Fix unused temps in >> >> GLMWizard>>#updateWhenPrevious >> GLMExpanderLabelNodeModel>>#elementColumn >> GLMListMorphicTest>>#testTextBackgroundColor >> GLMPortTest>>#testTransientValue" >> GLMPresentationTest>>#testCopyTheTransformation >> GLMTabulatorMorphicTest>>#testMultipleInitialSelection >> GLMTextMorphicTest>>#testResettingTheSelection >> GLMWizard>>#updatePane >> >> https://pharo.fogbugz.com/f/cases/20968/Unused-temps-in-various-GML-methods Issue URL: https://pharo.fogbugz.com/f/cases/20968 Unused temps in various GML methods PR URL: https://github.com/pharo-project/pharo/pull/680 Diff URL: https://github.com/pharo-project/pharo/pull/680/files Thanks to astares * 20967 Unused temps in CompiledMethodTest>>#testOrigin >> fix unsed temp var >> >> https://pharo.fogbugz.com/f/cases/20967/Unused-temps-in-CompiledMethodTest-t... Issue URL: https://pharo.fogbugz.com/f/cases/20967 Unused temps in CompiledMethodTest>>#testOrigin PR URL: https://github.com/pharo-project/pharo/pull/678 Diff URL: https://github.com/pharo-project/pharo/pull/678/files Thanks to astares * 20964 Unused temp variables in ConfigurationCommandLineHandler >> remove unused temp var >> >> https://pharo.fogbugz.com/f/cases/20964/Unused-temp-variables-in-Configurati... Issue URL: https://pharo.fogbugz.com/f/cases/20964 Unused temp variables in ConfigurationCommandLineHandler PR URL: https://github.com/pharo-project/pharo/pull/675 Diff URL: https://github.com/pharo-project/pharo/pull/675/files Thanks to astares * 20965 Unused temp variables in BaselineOfBasicTools and BaselineOfMorphic >> fix unsued temp variables Issue URL: https://pharo.fogbugz.com/f/cases/20965 Unused temp variables in BaselineOfBasicTools and BaselineOfMorphic PR URL: https://github.com/pharo-project/pharo/pull/676 Diff URL: https://github.com/pharo-project/pharo/pull/676/files Thanks to astares * 20966 Unused temps in Base64MimeConverterTest>>#testDecodeMimeHeader >> remove unused temp vars Issue URL: https://pharo.fogbugz.com/f/cases/20966 Unused temps in Base64MimeConverterTest>>#testDecodeMimeHeader PR URL: https://github.com/pharo-project/pharo/pull/677 Diff URL: https://github.com/pharo-project/pharo/pull/677/files Thanks to astares * 20921 PackageChecker>>check not needed anymore >> It is the only method in this class. We move this class to Deprecated70 to remove it in Pharo 8 >> >> https://pharo.fogbugz.com/f/cases/20921/PackageChecker-check-not-needed-anym... Issue URL: https://pharo.fogbugz.com/f/cases/20921 PackageChecker>>check not needed anymore PR URL: https://github.com/pharo-project/pharo/pull/679 Diff URL: https://github.com/pharo-project/pharo/pull/679/files Thanks to astares * 20963-Corrupted-SmartSuggestions-package >> https://pharo.fogbugz.com/f/cases/20963/Corrupted-SmartSuggestions-package Issue URL: https://pharo.fogbugz.com/f/cases/20963 PR URL: https://github.com/pharo-project/pharo/pull/673 Diff URL: https://github.com/pharo-project/pharo/pull/673/files Thanks to pavel-krivanek * 20971 - Class comment and method format fixes >> Removed invalid characters from Dictionary class comment. >> Removed '.' at the end of selectors. >> >> https://pharo.fogbugz.com/f/cases/20971/Class-comment-and-method-format-fixe... Issue URL: https://pharo.fogbugz.com/f/cases/20971 PR URL: https://github.com/pharo-project/pharo/pull/683 Diff URL: https://github.com/pharo-project/pharo/pull/683/files Thanks to theseion * 20969 Unused temp var in ZnRequestTests>>#testCookiesWriting >> https://pharo.fogbugz.com/f/cases/20969/ Issue URL: https://pharo.fogbugz.com/f/cases/20969 Unused temp var in ZnRequestTests>>#testCookiesWriting PR URL: https://github.com/pharo-project/pharo/pull/681 Diff URL: https://github.com/pharo-project/pharo/pull/681/files Thanks to astares * 20961 Broken Tonel file in FuelTests in pharo-project/pharo repo >> Fix Tonel format (Bug 20961) >> >> https://pharo.fogbugz.com/f/cases/resolve/20961/Broken-Tonel-file-in-FuelTes... Issue URL: https://pharo.fogbugz.com/f/cases/20961 Broken Tonel file in FuelTests in pharo PR URL: https://github.com/pharo-project/pharo/pull/670 Diff URL: https://github.com/pharo-project/pharo/pull/670/files Thanks to astares * 20960-Move-ExecutionDisplayPlugin-from-Reflectivity-Examples-directly-to-Nautilus- >> ExecutionDisplayPlugin and ExecutionDisplayMorph are moved to the Nautilus under 'Plugins-Examples' tag. >> >> https://pharo.fogbugz.com/f/cases/20960/Move-ExecutionDisplayPlugin-from-Ref... Issue URL: https://pharo.fogbugz.com/f/cases/20960 PR URL: https://github.com/pharo-project/pharo/pull/671 Diff URL: https://github.com/pharo-project/pharo/pull/671/files Thanks to dionisiydk * 20958-SmartSuggestions-should-not-depends-on-Nautilus >> SmartSuggestions do not depends on Nautilus anymore. >> Nautilus depends on it Issue URL: https://pharo.fogbugz.com/f/cases/20958 PR URL: https://github.com/pharo-project/pharo/pull/669 Diff URL: https://github.com/pharo-project/pharo/pull/669/files Thanks to dionisiydk * 20913 Gofer-Tests package cleanup (solve critiques) >> - use assert:equals: instead of = >> - use cascading where possible >> cosmetic changes only, no change of functionality Issue URL: https://pharo.fogbugz.com/f/cases/20913 Gofer PR URL: https://github.com/pharo-project/pharo/pull/638 Diff URL: https://github.com/pharo-project/pharo/pull/638/files Thanks to astares * Updated Fuel to 2.2.0 with the 64 bit adaptations >> https://pharo.manuscript.com/f/cases/20120/Fuel-is-not-64bits-ready Issue URL: https://pharo.fogbugz.com/f/cases/Updated Fuel to 2.2.0 with the 64 bit adaptations PR URL: https://github.com/pharo-project/pharo/pull/654 Diff URL: https://github.com/pharo-project/pharo/pull/654/files Thanks to theseion * 20879-ZnCrPortableWriteStreamnextPut-swallows-LFs >> 20879 ZnCrPortableWriteStream>>nextPut: swallows LFs >> >> - Fix ZnCrPortableWriteStream>>nextPut: so that Character lf is correctly written. >> - Add ZnCrPortableWriteStreamTests to confirm fix. >> >> Fogbugz: https://pharo.fogbugz.com/f/cases/20879 Issue URL: https://pharo.fogbugz.com/f/cases/20879 PR URL: https://github.com/pharo-project/pharo/pull/613 Diff URL: https://github.com/pharo-project/pharo/pull/613/files Thanks to akgrant43 * 20949 Use category "utilities" instead of "utils" in TheManifestBuilder >> "utilities" instead of "utils" recategorization >> >> https://pharo.fogbugz.com/f/cases/20949/Use-category-utilities-instead-of-ut... Issue URL: https://pharo.fogbugz.com/f/cases/20949 Use category "utilities" instead of "utils" in TheManifestBuilder PR URL: https://github.com/pharo-project/pharo/pull/665 Diff URL: https://github.com/pharo-project/pharo/pull/665/files Thanks to astares * 20938-Add-PrimitiveError-to-Pharo >> 20938-Add-PrimitiveError-to-Pharo >> >> Add PrimitiveError and update the SmalltalkImage special objects array to point to it. >> >> See: https://pharo.fogbugz.com/f/cases/20938/ Issue URL: https://pharo.fogbugz.com/f/cases/20938 PR URL: https://github.com/pharo-project/pharo/pull/660 Diff URL: https://github.com/pharo-project/pharo/pull/660/files Thanks to akgrant43 * 20946-Modifying-a-class-with-read-only-instances-should-raise-a-proper-error >> Modifying a class with read only instances should raise a proper error. >> When a class is modified and it has read-only instances the migration process is not possible. >> The class installer should report a proper error. >> Also all the subinstances should be checked. >> >> Issue: https://pharo.fogbugz.com/f/cases/20946/Modifying-a-class-with-read-only-ins... Issue URL: https://pharo.fogbugz.com/f/cases/20946 PR URL: https://github.com/pharo-project/pharo/pull/662 Diff URL: https://github.com/pharo-project/pharo/pull/662/files Thanks to tesonep * Add Shift class builder, Tonel and other improvements to Welcome changeLog >> https://pharo.fogbugz.com/f/cases/20955/Add-Shift-class-builder-Tonel-and-ot... Issue URL: https://pharo.fogbugz.com/f/cases/Add Shift class builder, Tonel and other improvements to Welcome changeLog PR URL: https://github.com/pharo-project/pharo/pull/668 Diff URL: https://github.com/pharo-project/pharo/pull/668/files Thanks to astares * 20947-NautilusRefactoring-package-should-not-depends-on-Nautilus-package >> It was almost all about method extensions which reference Nautils or NautilusUI but was packaged as NautilusRefactoring. >> So now they are extended directly by Nautilus package Issue URL: https://pharo.fogbugz.com/f/cases/20947 PR URL: https://github.com/pharo-project/pharo/pull/663 Diff URL: https://github.com/pharo-project/pharo/pull/663/files Thanks to dionisiydk * 20948-Remove-dependency-from-Nautilus-form-FT-examples >> Now actionIcon is constantly #scriptManagerIcon. >> >> https://pharo.fogbugz.com/f/cases/20948/Remove-dependency-from-Nautilus-form... Issue URL: https://pharo.fogbugz.com/f/cases/20948 PR URL: https://github.com/pharo-project/pharo/pull/664 Diff URL: https://github.com/pharo-project/pharo/pull/664/files Thanks to dionisiydk * 20945-Dependency-browser-should-not-depends-on-Nautilus >> All dependency to Nautilus is fixed. And all nautilus based names are renamed Issue URL: https://pharo.fogbugz.com/f/cases/20945 PR URL: https://github.com/pharo-project/pharo/pull/661 Diff URL: https://github.com/pharo-project/pharo/pull/661/files Thanks to dionisiydk * 20936-64bit-primitiveFailure-for-1-0-largeIdentityHash >> https://pharo.manuscript.com/f/cases/20936/64bit-primitiveFailure-for-1-0-la... >> >> Fixes on all identityHash and basicIdentityHash methods to make them work in 32 and 64 bits. >> Be careful not to create large integers Issue URL: https://pharo.fogbugz.com/f/cases/20936 PR URL: https://github.com/pharo-project/pharo/pull/658 Diff URL: https://github.com/pharo-project/pharo/pull/658/files Thanks to clementbera * 20934-Missing-implementation-for-supporting-read-only-onjects >> Added implementations and tests for read-only objects. A lot of methods do not throw an exception if a primitive fails through read-only-ness. Furthermore some assumptions about failing primitives are wrong in pharo and these have been corrected in order to handle properly a modification error. Made ModificationForbidden an exception instead of an error because it can be resumed. Issue URL: https://pharo.fogbugz.com/f/cases/20934 PR URL: https://github.com/pharo-project/pharo/pull/656 Diff URL: https://github.com/pharo-project/pharo/pull/656/files Thanks to noha * 20939-Updating-Shift-to-v1-0-2 >> Updating to the new version of Shift Class Builder and Installer (v1.0.2). >> >> Issue: https://pharo.manuscript.com/f/cases/20939 >> >> It fixes: >> >> - It copies the slots that have indexes in a better way. This fixes a problem with migrating instances when there is changes in different levels of the hierarchy. >> - Fixes a problem that was losing the category of class side methods when some changes where produced. >> - Adding tests for all these situations. >> - Extending the class installer to have an installation environment separate from the build environment. >> - Fixes also Issue 20882. https://pharo.fogbugz.com/f/cases/20882/New-class-installer-adds-anonymous-s... >> Issue URL: https://pharo.fogbugz.com/f/cases/20939 PR URL: https://github.com/pharo-project/pharo/pull/655 Diff URL: https://github.com/pharo-project/pharo/pull/655/files Thanks to tesonep * 16969-Use-a-Rectangle-instead-of-a-Margin >> redo the broken Stef's PR https://github.com/pharo-project/pharo/pull/601 Issue URL: https://pharo.fogbugz.com/f/cases/16969 PR URL: https://github.com/pharo-project/pharo/pull/650 Diff URL: https://github.com/pharo-project/pharo/pull/650/files Thanks to pavel-krivanek * Fix broken Tonel for #nonEmptyWithCopyNonIdentical >> All details in >> >> https://pharo.fogbugz.com/f/cases/20932/ Issue URL: https://pharo.fogbugz.com/f/cases/Fix broken Tonel for #nonEmptyWithCopyNonIdentical PR URL: https://github.com/pharo-project/pharo/pull/651 Diff URL: https://github.com/pharo-project/pharo/pull/651/files Thanks to astares * 20875-Alternative-dictionary-creation-method >> - Re-adding newFromKeys:andValues: to dictionary. >> - Adding it to SmallDictionary and OrderedDictionary. >> - Adding tests to such classes too. >> - All tests are green. Issue URL: https://pharo.fogbugz.com/f/cases/20875 PR URL: https://github.com/pharo-project/pharo/pull/617 Diff URL: https://github.com/pharo-project/pharo/pull/617/files Thanks to Ducasse * 20925 Super setUp need to be called in UnifiedFFI tests >> call super setUp in >> - FFICalloutMethodBuilderTest>>setUp >> - FFICalloutTests>>setUp >> - FFIExternalStructureFieldParserTests>>setUp >> >> and add some more class comments >> >> https://pharo.fogbugz.com/f/cases/20925/Super-setUp-need-to-be-called-in-Uni... Issue URL: https://pharo.fogbugz.com/f/cases/20925 Super setUp need to be called in UnifiedFFI tests PR URL: https://github.com/pharo-project/pharo/pull/645 Diff URL: https://github.com/pharo-project/pharo/pull/645/files Thanks to astares * 20926-Super-setUp-need-to-be-called-in-MCWorkingCopyBrowserTest >> call super setUp in setUp >> >> https://pharo.fogbugz.com/f/cases/20926/Super-setUp-need-to-be-called-in-MCw... Issue URL: https://pharo.fogbugz.com/f/cases/20926 PR URL: https://github.com/pharo-project/pharo/pull/646 Diff URL: https://github.com/pharo-project/pharo/pull/646/files Thanks to astares * 20924 Fix unused var tutorial in TutorialPlayerTestWithMockTutorial>>setUp >> make use of tutorial variable Issue URL: https://pharo.fogbugz.com/f/cases/20924 Fix unused var tutorial in TutorialPlayerTestWithMockTutorial>>setUp PR URL: https://github.com/pharo-project/pharo/pull/644 Diff URL: https://github.com/pharo-project/pharo/pull/644/files Thanks to astares * 20916 Use category "utilities" instead of "utils" in Kernel classes >> utils -> utilities >> >> only recategorization, no change in behavior Issue URL: https://pharo.fogbugz.com/f/cases/20916 Use category "utilities" instead of "utils" in Kernel classes PR URL: https://github.com/pharo-project/pharo/pull/640 Diff URL: https://github.com/pharo-project/pharo/pull/640/files Thanks to astares * Cleanup of SimpleButtonMorph newWithLabel to have a yourself >> add missing yourself >> (the brace is not necessary but for better readability according to BPP) >> >> >> https://pharo.fogbugz.com/f/cases/20909/SimpleButtonMorph-newWithLabel-shoul... Issue URL: https://pharo.fogbugz.com/f/cases/Cleanup of SimpleButtonMorph newWithLabel to have a yourself PR URL: https://github.com/pharo-project/pharo/pull/635 Diff URL: https://github.com/pharo-project/pharo/pull/635/files Thanks to astares * 20927 - Super setUp need to be called in FileSystem-Tests >> call super setUp in >> - FileReferenceTest >> - FileSystemHandleTest >> - FileSystemResolverTest >> - FileSystemTest >> >> https://pharo.fogbugz.com/f/cases/20927/Super-setUp-need-to-be-called-in-Fil... Issue URL: https://pharo.fogbugz.com/f/cases/20927 PR URL: https://github.com/pharo-project/pharo/pull/647 Diff URL: https://github.com/pharo-project/pharo/pull/647/files Thanks to astares * 20920 RPackageOrganizer should use packageClass instead of RPackage references >> https://pharo.fogbugz.com/f/cases/20920/RPackageOrganizer-should-use-package... Issue URL: https://pharo.fogbugz.com/f/cases/20920 RPackageOrganizer should use packageClass instead of RPackage references PR URL: https://github.com/pharo-project/pharo/pull/648 Diff URL: https://github.com/pharo-project/pharo/pull/648/files Thanks to astares * 20911 Cleanup MCClassDefinitionTest >> - categorize methods >> - use assert:equal: instead of = >> >> no change in behavior >> >> https://pharo.fogbugz.com/f/cases/20911/Cleanup-MCClassDefinitionTest Issue URL: https://pharo.fogbugz.com/f/cases/20911 Cleanup MCClassDefinitionTest PR URL: https://github.com/pharo-project/pharo/pull/637 Diff URL: https://github.com/pharo-project/pharo/pull/637/files Thanks to astares * 20918 Super setUp need to be called in Collections-Tests >> call super setUp in setUp method Issue URL: https://pharo.fogbugz.com/f/cases/20918 Super setUp need to be called in Collections PR URL: https://github.com/pharo-project/pharo/pull/642 Diff URL: https://github.com/pharo-project/pharo/pull/642/files Thanks to astares * 20922-Update-Iceberg-to-v066 >> https://pharo.fogbugz.com/f/cases/20922/Update-Iceberg-to-v0-6-6 >> >> Update Iceberg to v0.6.6 Issue URL: https://pharo.fogbugz.com/f/cases/20922 PR URL: https://github.com/pharo-project/pharo/pull/643 Diff URL: https://github.com/pharo-project/pharo/pull/643/files Thanks to pavel-krivanek * Categorize methods in MCMczInstallerTest >> https://pharo.fogbugz.com/f/cases/20910/Categorize-methods-in-MCMczInstaller... Issue URL: https://pharo.fogbugz.com/f/cases/Categorize methods in MCMczInstallerTest PR URL: https://github.com/pharo-project/pharo/pull/636 Diff URL: https://github.com/pharo-project/pharo/pull/636/files Thanks to astares * 20917-Super-setUp-need-to-be-called-in-Debugger-Tests >> call super setUp >> >> https://pharo.fogbugz.com/f/cases/20917/Super-setUp-need-to-be-called-in-Deb... Issue URL: https://pharo.fogbugz.com/f/cases/ 20917 PR URL: https://github.com/pharo-project/pharo/pull/641 Diff URL: https://github.com/pharo-project/pharo/pull/641/files Thanks to astares * 20914-remove-remaining-halt >> remove halt. >> >> fixes issue #20914 Issue URL: https://pharo.fogbugz.com/f/cases/20914 PR URL: https://github.com/pharo-project/pharo/pull/639 Diff URL: https://github.com/pharo-project/pharo/pull/639/files Thanks to estebanlm * 20891-Changesorter-should-prompt-with-a-file-dialog >> first version for https://pharo.fogbugz.com/f/cases/20891/Changesorter-should-prompt-with-a-fi... Issue URL: https://pharo.fogbugz.com/f/cases/20891 PR URL: https://github.com/pharo-project/pharo/pull/628 Diff URL: https://github.com/pharo-project/pharo/pull/628/files Thanks to Ducasse * 20899-Add-comments-to-Color-methods >> Adding comments to instance method and introducing more protocols + removing broken code from class comments. >> https://pharo.fogbugz.com/f/cases/20899/Add-comments-to-Color-methods Issue URL: https://pharo.fogbugz.com/f/cases/20899 PR URL: https://github.com/pharo-project/pharo/pull/633 Diff URL: https://github.com/pharo-project/pharo/pull/633/files Thanks to Ducasse * 20823-sumNumbers-without-block-is-missing-for-collections >> sumNumbers is added as missing handy function to sum array items which are numbers Issue URL: https://pharo.fogbugz.com/f/cases/20823 PR URL: https://github.com/pharo-project/pharo/pull/619 Diff URL: https://github.com/pharo-project/pharo/pull/619/files Thanks to dionisiydk * 20797-Exception-class-missing-a-method >> Adding a new constructor to exception to include the tag. >> And adding a test for it. Issue URL: https://pharo.fogbugz.com/f/cases/20797 PR URL: https://github.com/pharo-project/pharo/pull/622 Diff URL: https://github.com/pharo-project/pharo/pull/622/files Thanks to tesonep * 20888-printOndelimiter-is-broken >> https://pharo.fogbugz.com/f/cases/20888/printOn-delimiter-is-broken >> >> write the delimiter as string, not as printString Issue URL: https://pharo.fogbugz.com/f/cases/20888 PR URL: https://github.com/pharo-project/pharo/pull/626 Diff URL: https://github.com/pharo-project/pharo/pull/626/files Thanks to pavel-krivanek * 20694-Do-we-need-Objecthead-and-Objecttail- >> Renamed the #head and #tail methods injected by the Morphic-Base package into the Object and Association class to #treeNodeHead and #treeNodeTail. >> Reflected the change to the code of the MorphTreeNodeMorph class that was using these two methods. Issue URL: https://pharo.fogbugz.com/f/cases/20694 PR URL: https://github.com/pharo-project/pharo/pull/623 Diff URL: https://github.com/pharo-project/pharo/pull/623/files Thanks to dupriezt * 19913-Nautilus-Class-pane-have-a-weird-behavior-when-we-hold-shift-while-clicking-on-classes >> >> Fixing the nautilus class selection to work properly in both scenarios, when selecting only one or multiple. >> >> https://pharo.fogbugz.com/f/cases/19913 Issue URL: https://pharo.fogbugz.com/f/cases/19913 PR URL: https://github.com/pharo-project/pharo/pull/625 Diff URL: https://github.com/pharo-project/pharo/pull/625/files Thanks to tesonep * 20886-Undeclared-RBMethodHasNoTimeStampRule- >> https://pharo.fogbugz.com/f/cases/20886/Undeclared-RBMethodHasNoTimeStampRul... >> >> remove undeclared reference Issue URL: https://pharo.fogbugz.com/f/cases/20886 PR URL: https://github.com/pharo-project/pharo/pull/624 Diff URL: https://github.com/pharo-project/pharo/pull/624/files Thanks to pavel-krivanek * 20817-Disable-or-remove-RBMethodHasNoTimeStampRule >> https://pharo.fogbugz.com/f/cases/20817/ >> >> remove RBMethodHasNoTimeStampRule Issue URL: https://pharo.fogbugz.com/f/cases/20817 PR URL: https://github.com/pharo-project/pharo/pull/620 Diff URL: https://github.com/pharo-project/pharo/pull/620/files Thanks to pavel-krivanek * 20813-Improve-comments-of-ifNotNil-and-ifNotNilifNil >> https://pharo.fogbugz.com/f/cases/20813/ >> >> ifNil.cs by Stef Issue URL: https://pharo.fogbugz.com/f/cases/20813 PR URL: https://github.com/pharo-project/pharo/pull/618 Diff URL: https://github.com/pharo-project/pharo/pull/618/files Thanks to pavel-krivanek * 20861-split-and-friends-should-have-better-comments >> https://pharo.fogbugz.com/f/cases/20861/split-and-friends-should-have-better... Issue URL: https://pharo.fogbugz.com/f/cases/20861 PR URL: https://github.com/pharo-project/pharo/pull/611 Diff URL: https://github.com/pharo-project/pharo/pull/611/files Thanks to Ducasse * 20880-failing-test-testIfCompletelyBroken >> https://pharo.fogbugz.com/f/cases/20880/failing-test-testIfCompletelyBroken >> fix the exected numbers Issue URL: https://pharo.fogbugz.com/f/cases/20880 PR URL: https://github.com/pharo-project/pharo/pull/614 Diff URL: https://github.com/pharo-project/pharo/pull/614/files Thanks to pavel-krivanek * 20876-Drag-and-drop-of-methods-is-broken >> Nautilus widgets should return TransferPresenter instead of TransferMorph because Nautilus uses data source of Spec component which supposed to work with presenters. >> It fixes drag and drop in Nautilus. >> >> AbstractWidgetPresenter fix in wants drop check. It allow only #isTransferable draggedMorphs. It fixes general drag of windows over spec based fast tables . >> >> https://pharo.manuscript.com/f/cases/20876/Drag-and-drop-of-methods-is-broke... Issue URL: https://pharo.fogbugz.com/f/cases/20876 PR URL: https://github.com/pharo-project/pharo/pull/616 Diff URL: https://github.com/pharo-project/pharo/pull/616/files Thanks to dionisiydk * 15473-Float-round-to-n-decimal-places-is-not-in-agreement-with-printShowingDecimalPlaces >> This is issue: >> https://pharo.fogbugz.com/f/cases/15473/Float-round-to-n-decimal-places-is-n... >> >> It also fixes: >> https://pharo.fogbugz.com/f/cases/15472/Fraction-rounding-to-n-decimal-place... >> https://pharo.fogbugz.com/f/cases/15471/Can-t-round-Float-fmax-to-2-decimal-... >> >> Note: the new implementation does round exact tie away from zero exactly like #round do. >> It is also in agreement with printShowingDecimalPlaces: >> >> If you wish to roundToNearestTieToEven, then please open a new issue >> (but it will be better to also modify printShowingDecimalPlaces: then) >> >> The new implementation is a bit slower but it is correct. >> >> It was proposed to deprecate round: in >> >> I’m not so sure - round: can be useful for monetary apps and is available in other languages (Python, etc…) >> banker rounding (tie to even) might be a better option though. >> >> Last, I’ve replaced the `< expr: foo result: bar>` annotation with `“ foo >>> bar “` >> The reason is that the expr:result: form only works for a literal result which is SEVERLY limited. Issue URL: https://pharo.fogbugz.com/f/cases/15473 PR URL: https://github.com/pharo-project/pharo/pull/514 Diff URL: https://github.com/pharo-project/pharo/pull/514/files Thanks to nicolas-cellier-aka-nice * 20872-WidgetExamples-classexampleDialogs-is-broken-in-Pharo-7 >> 20872 WidgetExamples class>>exampleDialogs is broken in Pharo 7 >> >> See: https://pharo.fogbugz.com/f/cases/20872/WidgetExamples-class-exampleDialogs-... Issue URL: https://pharo.fogbugz.com/f/cases/20872 PR URL: https://github.com/pharo-project/pharo/pull/610 Diff URL: https://github.com/pharo-project/pharo/pull/610/files Thanks to akgrant43 * 20796-String-should-understand-readStreamDowriteStreamDo-to-be-polymorph-with-FileReference >> Add #readStreamDo: and #writeStreamDo: to SequeanceableCollection with a comment and a test in StringTest >> >> Case 20796 : String should understand #readStreamDo:/#writeStreamDo: to be polymorph with FileReference Issue URL: https://pharo.fogbugz.com/f/cases/20796 PR URL: https://github.com/pharo-project/pharo/pull/605 Diff URL: https://github.com/pharo-project/pharo/pull/605/files Thanks to jecisc * 20808 createRepositoryURLtoNameMappings needs to be more flexible >> https://pharo.fogbugz.com/f/cases/20808/createRepositoryURLtoNameMappings-ne... Issue URL: https://pharo.fogbugz.com/f/cases/20808 createRepositoryURLtoNameMappings needs to be more flexible PR URL: https://github.com/pharo-project/pharo/pull/587 Diff URL: https://github.com/pharo-project/pharo/pull/587/files Thanks to astares * 20869-Add-tooltips-to-world-menu-entries >> - Add help to every menu entry of the world menu >> - Be sur every help begin with an uppercase and end with a dot >> - Add a release test to be sure we do not integrate a tool without help menu later >> >> Case 20869 : https://pharo.fogbugz.com/f/cases/20869/Add-tooltips-to-world-menu-entries >> >> /!\/!\/!\ To integrate after https://github.com/pharo-vcs/iceberg/pull/538 for the release test /!\/!\/!\ Issue URL: https://pharo.fogbugz.com/f/cases/20869 PR URL: https://github.com/pharo-project/pharo/pull/606 Diff URL: https://github.com/pharo-project/pharo/pull/606/files Thanks to jecisc * 20864-add-examples-to-SequenceableCollectionaggregateRuns >> rename aggregateRuns: to groupByRuns: and add some examples. Deprecate the coriginal method >> >> https://pharo.fogbugz.com/f/cases/20864/add-examples-to-SequenceableCollecti... Issue URL: https://pharo.fogbugz.com/f/cases/20864 PR URL: https://github.com/pharo-project/pharo/pull/607 Diff URL: https://github.com/pharo-project/pharo/pull/607/files Thanks to pavel-krivanek * 20836-Cleaning-comments >> https://pharo.fogbugz.com/f/cases/20836/Cleaning-comments Issue URL: https://pharo.fogbugz.com/f/cases/20836 PR URL: https://github.com/pharo-project/pharo/pull/595 Diff URL: https://github.com/pharo-project/pharo/pull/595/files Thanks to Ducasse * 20870-Spec-FastTable-badly-support-drag-and-drop >> missing methods are added. >> I tested them in Commander-Examples Issue URL: https://pharo.fogbugz.com/f/cases/20870 PR URL: https://github.com/pharo-project/pharo/pull/608 Diff URL: https://github.com/pharo-project/pharo/pull/608/files Thanks to dionisiydk * 20684-Categorizer-should-categorize-reset-methods-in-initialization-protocol >> Categorizer should categorize #reset methods in 'initialization' protocol. >> >> ML subject: http://forum.world.st/Convention-for-quot-reset-quot-methods-td5019493.html >> >> Resolve case 20684 : Categorizer should categorize #reset methods in 'initialization' protocol Issue URL: https://pharo.fogbugz.com/f/cases/20684 PR URL: https://github.com/pharo-project/pharo/pull/604 Diff URL: https://github.com/pharo-project/pharo/pull/604/files Thanks to jecisc * 20859-Update-Hermes-to-detect-new-Undeclared-and-overwritten-methods-and-classes >> Integrating the version 2.3.0 that detects if new Undeclared are created when loading a package and if methods and classes are overwritten. >> >> This is to detect no well resolved dependencies during the bootstrap. >> >> https://pharo.fogbugz.com/f/cases/20859/Update-Hermes-to-detect-new-Undeclar... Issue URL: https://pharo.fogbugz.com/f/cases/20859 PR URL: https://github.com/pharo-project/pharo/pull/597 Diff URL: https://github.com/pharo-project/pharo/pull/597/files Thanks to tesonep * 20832-boostrap-process-should-use-IceProxyMCVersionInfo-instead-system-repositories >> fix case #20832 >> >> THIS CHANGES THE CONTRIBUTION PROCESS SO DO NOT INTEGRATE UNTIL I HAVE TIME TO EXPLAIN IN LISTS ;) Issue URL: https://pharo.fogbugz.com/f/cases/20832 PR URL: https://github.com/pharo-project/pharo/pull/584 Diff URL: https://github.com/pharo-project/pharo/pull/584/files Thanks to estebanlm * 20680-Add-comments-to-FileSystem-packages >> Add packages comments to FileSystem packages. >> >> See case 20680 : https://pharo.fogbugz.com/f/cases/20680/Add-comments-to-FileSystem-packages >> >> Please, review them. My english is far from been good :) Issue URL: https://pharo.fogbugz.com/f/cases/20680 PR URL: https://github.com/pharo-project/pharo/pull/603 Diff URL: https://github.com/pharo-project/pharo/pull/603/files Thanks to jecisc * 20498-Replace-actual-Class-Installer-with-a-newer-modular-Version >> New try to integrate this PR. I had to reconstruct it because of migrating to Tonel. >> >> Still I didn't implemented the comments of Pavel, I will do when I got to the same state I was in the old PR. >> >> https://github.com/pharo-project/pharo/pull/334/files >> >> https://pharo.fogbugz.com/f/cases/20498/Replace-actual-Class-Installer-with-... Issue URL: https://pharo.fogbugz.com/f/cases/20498 PR URL: https://github.com/pharo-project/pharo/pull/585 Diff URL: https://github.com/pharo-project/pharo/pull/585/files Thanks to tesonep * 20865-update-iceberg-to-065 >> using iceberg 0.6.5 >> >> fix case #20865 Issue URL: https://pharo.fogbugz.com/f/cases/20865 PR URL: https://github.com/pharo-project/pharo/pull/599 Diff URL: https://github.com/pharo-project/pharo/pull/599/files Thanks to estebanlm * 20866-revert-retryTimes-parameter-on-CI >> https://pharo.fogbugz.com/f/cases/20866/revert-retryTimes-parameter-on-CI Issue URL: https://pharo.fogbugz.com/f/cases/20866 PR URL: https://github.com/pharo-project/pharo/pull/600 Diff URL: https://github.com/pharo-project/pharo/pull/600/files Thanks to pavel-krivanek * 20773 - Unused temps in BlockClosure value, value:, value:value:, ... >> Remove unused temps from >> >> BlockClosure>>#value:value:value:value: >> BlockClosure>>#value:value:value: >> BlockClosure>>#value >> BlockClosure>>#value: >> BlockClosure>>#value:value: Issue URL: https://pharo.fogbugz.com/f/cases/20773 PR URL: https://github.com/pharo-project/pharo/pull/558 Diff URL: https://github.com/pharo-project/pharo/pull/558/files Thanks to astares * 20860-Cursor-webLink-is-initialized-wrongly-and-transparent >> https://pharo.manuscript.com/f/cases/20860 >> >> set weblink cursor shape Issue URL: https://pharo.fogbugz.com/f/cases/20860 PR URL: https://github.com/pharo-project/pharo/pull/598 Diff URL: https://github.com/pharo-project/pharo/pull/598/files Thanks to pavel-krivanek * 20857-Confirm-dialog-return-nil-when-closed-by-cmd-w >> https://pharo.manuscript.com/f/cases/20857 >> >> fix by Henrik Nergaard Issue URL: https://pharo.fogbugz.com/f/cases/20857 PR URL: https://github.com/pharo-project/pharo/pull/596 Diff URL: https://github.com/pharo-project/pharo/pull/596/files Thanks to pavel-krivanek