Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #3186: Enable Lens DF to use Reference DF as attribute field #3187

Open
wants to merge 9 commits into
base: rel-11_0
Choose a base branch
from

Conversation

stefanhaerter
Copy link
Contributor

Also another small bugfix and a readability enhancement.

Referencing #3186.

@stefanhaerter stefanhaerter force-pushed the issue-#3186-setsdynamiccontent branch 2 times, most recently from 0caa8f1 to 40411ed Compare March 19, 2024 12:56
@@ -289,7 +289,7 @@ sub ObjectDataGet {
Param => 'TicketID',
);

return if !$TicketID;
return unless $TicketID;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool, ich bin ein Fan von 'return unless'

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You got me hooked on that 😁

@stefanhaerter stefanhaerter force-pushed the issue-#3186-setsdynamiccontent branch 2 times, most recently from 211644a to 11feedc Compare June 3, 2024 09:21
@stefanhaerter stefanhaerter force-pushed the issue-#3186-setsdynamiccontent branch 2 times, most recently from 31acbe5 to 01ab40a Compare June 10, 2024 13:32
@stefanhaerter stefanhaerter force-pushed the issue-#3186-setsdynamiccontent branch 2 times, most recently from e82c7c7 to 081f30a Compare September 30, 2024 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants