November 2016 - Pharo-dev - lists.pharo.org

why RBComment is not called RBCommentNode
by stepharo Nov. 4, 2016

Nov. 4, 2016

Hi marcus et al why RBComment is not called RBCommentNode and why it is not a subclass of RBNode? Stef

5 13

[pharo-project/pharo-core]
by GitHub Nov. 4, 2016

Nov. 4, 2016

Branch: refs/tags/60278 Home: https://github.com/pharo-project/pharo-core

1 0

[pharo-project/pharo-core] b3a92e: 60278
by GitHub Nov. 4, 2016

Nov. 4, 2016

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: b3a92e10b66a76d796c1ae4643aab2295a429845 https://github.com/pharo-project/pharo-core/commit/b3a92e10b66a76d796c1ae46… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-11-04 (Fri, 04 Nov 2016) Changed paths: M GT-Debugger.package/GTDebuggerSystemSettings.class/class/settings/debugSettingsOn_.st M GT-EventRecorder.package/GTEventRecorderSettings.class/class/settings/urlSettingsOn_.st M GT-Playground.package/GTPlayground.class/class/settings/settingsOn_.st M GT-Playground.package/GTUrlProvider.class/class/settings/settingsOn_.st M Growl.package/GrowlMorph.class/class/settings/growlSettingsOn_.st M Morphic-Core.package/Morph.class/instance/dropping%2Fgrabbing/formerPosition_.st M OSWindow-Core.package/OSTouchHoldDetector.class/class/settings/maxDistanceTravelledSettingsOn_.st M OSWindow-Core.package/OSTouchHoldDetector.class/class/settings/waitingTimeSettingsOn_.st M OSWindow-Core.package/OSTouchOneFingerDoubleTapDetector.class/class/settings/doubleTouchSpeedSettingsOn_.st M OSWindow-Core.package/OSTouchOneFingerDoubleTapDetector.class/class/settings/maxDistanceSettingsOn_.st M OSWindow-Core.package/OSTouchPinchDetector.class/class/settings/maxDistanceBetweenFingersLineSettingsOn_.st M OSWindow-Core.package/OSTouchPinchDetector.class/class/settings/minDistanceBeforeDetectionSettingsOn_.st M OSWindow-Core.package/OSTouchRotationDetector.class/class/settings/minAngleBeforeDetectionSettingsOn_.st M OSWindow-Core.package/OSTouchScrollDetector.class/class/settings/minDistanceBeforeDetectionSettingsOn_.st M OSWindow-Core.package/OSTouchScrollDetector.class/class/settings/persistentDirectionSettingsOn_.st M OSWindow-Core.package/OSTouchSwipeDetector.class/class/settings/minDistToBorderSettingsOn_.st M OSWindow-Core.package/OSTouchSwipeDetector.class/class/settings/minDistanceBeforeDetectionSettingsOn_.st M Renraku.package/ReSettings.class/class/settings-definition/reGroupSettingsOn_.st M Renraku.package/ReSettings.class/class/settings-definition/showErrorsSettingsOn_.st M Renraku.package/ReSettings.class/class/settings-definition/spotterPluginSettingsOn_.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60277.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60278.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60277.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60278.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M ShoreLine-Report-Settings.package/ShoreLineReportSettings.class/class/settings/reportFlagSettingsOn_.st M ShoreLine-Report-Settings.package/ShoreLineReportSettings.class/class/settings/reportServerSettingsOn_.st A System-Settings-Rules.package/SettingDontTranslateDescriptionRule.class/README.md A System-Settings-Rules.package/SettingDontTranslateDescriptionRule.class/definition.st A System-Settings-Rules.package/SettingDontTranslateDescriptionRule.class/instance/accessing/group.st A System-Settings-Rules.package/SettingDontTranslateDescriptionRule.class/instance/accessing/name.st A System-Settings-Rules.package/SettingDontTranslateDescriptionRule.class/instance/helpers/checkPreconditionOn_.st A System-Settings-Rules.package/SettingDontTranslateDescriptionRule.class/instance/initialization/initialize.st A System-Settings-Rules.package/SettingDontTranslateDescriptionRule.class/instance/running/check_forCritiquesDo_.st Log Message: ----------- 60278 19288 Sending translated should not be the responsibility of the client of a Setting - description: case https://pharo.fogbugz.com/f/cases/19288 19298 Rule for settings description translation https://pharo.fogbugz.com/f/cases/19298 http://files.pharo.org/image/60/60278.zip

1 0

About setProperty:toValue: vs. valueOfProperty:
by stepharo Nov. 4, 2016

Nov. 4, 2016

I would like to be able to compare the **arguments** passed to setProperty:toValue: and the ones of valueOfProperty:, valueOfProperty:ifAbsent:, valueOfProperty:ifAbsentPut:, and hasProperty: I quite sure that we will discover more deadcode in morphic. Any ideas? PS: I hope (but I cannot read the code of Bloc because it crashes my vm at startup) that Bloc and Brick will not use such patterns. There are plague and lead to shitty and crappy code. Stef

3 3

the kind of code I would like to avoid....
by stepharo Nov. 3, 2016

Nov. 3, 2016

I do not get why a property is useful here and why not just a temp is not good enough. Especially since there are no other users of the property. displayFiltered: evt | matchStr matches feedbackMorph | matchStr := self valueOfProperty: #matchString. matches := self menuItems select: [:m | | isMatch | isMatch := matchStr isEmpty or: [ m contents includesSubstring: matchStr caseSensitive: false ]. m isEnabled: isMatch. isMatch ]. feedbackMorph := self valueOfProperty: #feedbackMorph. feedbackMorph ifNil: [ feedbackMorph := TextMorph new autoFit: true; color: Color darkGray. self addLine; addMorphBack: feedbackMorph lock. self setProperty: #feedbackMorph toValue: feedbackMorph. self fullBounds. "Lay out for submorph adjacency"]. feedbackMorph contents: '<', matchStr, '>'. matchStr isEmpty ifTrue: [ feedbackMorph delete. self lastSubmorph delete. self removeProperty: #feedbackMorph ]. " This method is invoked with evt = nil from MenuMorph >> removeMatchString. The current implementation can't select an item without an event. (Yet :D)" (evt notNil and: [ matches size >= 1 ]) ifTrue: [self selectItem: matches first event: evt]

1 0

shoreline, GT, renaku dev please listen
by stepharo Nov. 3, 2016

Nov. 3, 2016

Hi guys Please read the following https://pharo.fogbugz.com/f/cases/19288/Sending-translated-should-not-be-th… There is no need to do description: 'blo' translated in setting def. because it is already done internally (how it should be). I hope that yuriy will harvest this nice rule. I browsed the 1099 users of description:/translated and fixed the ones that needed to be fixed. Can you pay attention to merge back in your stream? Stef

1 0

TheManifestBuilder>>#packageNameOf:
by stepharo Nov. 3, 2016

Nov. 3, 2016

Still a name around https://pharo.fogbugz.com/f/cases/19282/Still-name-used The method Object>>#name called from TheManifestBuilder>>#packageNameOf: has been deprecated. Implement your own domain representation of an object, or use #asString or #printString instead. Select Proceed to continue, or close this window to cancel the operation. Stef

1 0

Re: [Pharo-dev] [Seaside-dev] Build failures
by Sven Van Caekenberghe Nov. 3, 2016

Nov. 3, 2016

It already got reported twice, there are issues for it, but so far no luck (i.e. no actual resolution). I would guess that before the linux os of the CI slaves ran in 32-bit, while now it got upgraded to 64-bit. CC-ing pharo-dev > On 28 Oct 2016, at 14:12, Johan Brichau <johan(a)inceptive.be> wrote: > > Does anybody have any idea why this is failing: > >> /pharo: line 11: <https://ci.inria.fr/pharo-contribution/job/Grease/PHARO=40,VERSION=stable,V…>: No such file or directory > > Did not look at it yet, but it seems like something changed on de CI server?? > > Johan_______________________________________________ > seaside-dev mailing list > seaside-dev(a)lists.squeakfoundation.org > http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev

2 4

doing a backend change in pharo.org. It will be down for 1h.
by Esteban Lorenzano Nov. 3, 2016

Nov. 3, 2016

subj

1 0

[pharo-project/pharo-core]
by GitHub Nov. 3, 2016

Nov. 3, 2016

Branch: refs/tags/60277 Home: https://github.com/pharo-project/pharo-core

1 0