[Seaside-dev] Configurable current request context variable

Esteban A. Maringolo emaringolo at gmail.com
Tue May 16 20:24:42 UTC 2017


2017-05-16 17:20 GMT-03:00 Max Leske <maxleske at gmail.com>:
> Hi Philippe,
>
> I’ve created an implementation, trying to do what you suggest; these are the changes:
>
> - removed WADynamicVariable
> - added GRDynamicVariable to Grease-Core
> - subclasses of WADynamicVariable now inherit from GRDynamicVariable
> - test cases for WADynamicVariable moved to Grease-Tests-Core (not sure about this. The tests should be the same for all platforms but Grease-Core does not contain GRDynamicVariable…)

Great! Thank you Max.

Is Grease-*Dialect*-Core (e.g. Grease-Pharo-Core) a prerequisite of Grease-Core?


> I don’t have write permissions on STHub, so I’m attaching the patches below.

As expected, the file didn't make it to the attachments of the mail :)

> I can also make the change for Squeak but I’m not sure how I would go about committing the change for VW / VASt. Let me know what you think.




Esteban A. Maringolo


More information about the seaside-dev mailing list