Fix getting correct catalog - #2367
Conversation
|
I don't understand why it's named after static assets. What's the reasoning behind the naming? Also this concept of setting the catalog based on operating on a catalog discriminated entity probably should be done deeper in the admin flow. More specifically, this probably needs to be done closer to persistence in some way. This is just one of likely many entry points to creating a relationship. |
|
@cja769 I pushed changes, but not sure where could put trigger for my handler |
|
@oleksiimiroshnyk I tested these changes and it's hitting the extension manager and everything but the new piece in the BroadleafAdminRequestProcessor doesn't seem to be getting hit at all. I checked from the client side and I'm not seeing the parameter in the request. |
|
@cja769 are you sure you have everything up to date, rebuild etc ? As a side effect, now all status, ribbon etc calls are sent with that parameter. |
|
@cja769 found why you don't see it. |
a0ee4d2 to
8465052
Compare
… be saved in the wrong catalog in certain situations when a standard site has multiple catalog
8465052 to
0b84e97
Compare

A Brief Overview
Fix getting correct catalog when standard site has multiple catalogs
Link to QA issue
BroadleafCommerce/QA#4110