fflib_QueryFactory().selectFieldSet() fails with when FieldSet queries namespaced fields about fflib-apex-common HOT 10 CLOSED

@capeterson one for you? 👍

from fflib-apex-common.

Looks like it. I'll dig in shortly and see what can be done.
On Aug 13, 2014 3:52 AM, "Andrew Fawcett" [email protected] wrote:

@capeterson https://github.com/capeterson one for you? [image: 👍]

—
Reply to this email directly or view it on GitHub
#24 (comment)
.

from fflib-apex-common.

This might be connected to this Know Issue: https://success.salesforce.com/issues_view?id=a1p30000000SzK3AAK

from fflib-apex-common.

@afawcett from looking at the class I think the root cause lies out side this class and instead in fflib_SObjectDescribe

from fflib-apex-common.

@afawcett and @capeterson I hope I could fix this in pull request #25

from fflib-apex-common.

Taking a look at this I have to ask if it actually fixed the issue for you? The fflib_SObjectDescribe.getField method delegates to the FieldsMap.get implementation which already lower-cases the input at that level. Although of course it's late and I may be missing some detail.

from fflib-apex-common.

It's true that you store the field names in lower case in your Describe cache class, but you definitly do not lower case the field names you extract from a Fieldsets FieldSetMember.

FieldSetMember myField = ...
String stillMixedCase = myField.getFieldPath();
...getFieldToken(stillMixedCase);
Schema.DescribeSObjectResult describe = getDescribe();
Schema.SObjectField field = describe.getField(stillMixedCase);

from fflib-apex-common.

Hi @up2go-rsoesemann is there still an issue here? @capeterson whats your thoughts?

from fflib-apex-common.

If it's still an issue certainly needs to be fixed, I also need to reconcile the private and public versions, I think we've had a few bug fixes on the internal version I need to migrate to this repo.

from fflib-apex-common.

Old and possibly fixed?

from fflib-apex-common.