search

struts-community-plugins / struts2-jquery

Struts2 jQuery Plugin
Apache License 2.0
83 stars 49 forks source link

Avoids setting ID if form ID is null #480

Closed lukaszlenart closed 2 weeks ago

lukaszlenart commented 4 months ago

Closes #465

lukaszlenart commented 4 months ago

@nikosdimitrakas this can potentially break #190

nikosdimitrakas commented 4 months ago

I don't see why the null checks would cause any problems. I can test your changes once they are part of a snapshot.

lukaszlenart commented 4 months ago

I don't see why the null checks would cause any problems. I can test your changes once they are part of a snapshot.

There is a longer discussion and as I'm not using the plugin I have no idea if the change is safe