Skip to content

Detect property names at the beginning of events - #9536

Closed
cragkhit wants to merge 1 commit into
apache:masterfrom
cragkhit:matcha-detect-leading-property-names
Closed

Detect property names at the beginning of events#9536
cragkhit wants to merge 1 commit into
apache:masterfrom
cragkhit:matcha-detect-leading-property-names

Conversation

@cragkhit

@cragkhit cragkhit commented Aug 1, 2026

Copy link
Copy Markdown

Hi. We are researchers from Mahidol University, Thailand, and the State University of Ceará, Brazil, working on a research project for improving open-source projects by using the latest accepted answer from Stack Overflow that matched your code snippet. We found this recommendation for improving your code from https://stackoverflow.com/a/6702420.

Note: Our study is approved by the Institutional Review Board of Mahidol University. You can find the participant information sheet explaining this study https://drive.google.com/file/d/1ml5AqrtWQ9pnifTQyTFTcWQmwp6RuPA7/view?usp=sharing.


Proposed change

Allowed matching web descriptor property names at index zero to trigger the expected view refresh.

Allowed matching web descriptor property names at index zero to trigger the expected view refresh.
@matthiasblaesing

Copy link
Copy Markdown
Contributor

Please don't just state who are and why you do things, the link to a random google drive looks like a bad scamming test.

The linked stack overflow has nothing to do with the touched area of code.

What is critically missing here is: what problem is fixed here and why is that the right approach.

@cragkhit cragkhit closed this Aug 2, 2026
@cragkhit

cragkhit commented Aug 2, 2026

Copy link
Copy Markdown
Author

@matthiasblaesing You're right. I'm sorry about this. The patch is auto-generated by our research tool and we just discovered the issue that it made irrelevant patch without basing on the code on Stack Overflow. We'll be more careful next time. I have now closed this PR.

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