Skip to content

Navigation Menu

Appearance settings

View all features
- BY COMPANY SIZE
  Enterprises
  Small and medium teams
  Startups
  Nonprofits
- BY USE CASE
  App Modernization
  DevSecOps
  DevOps
  CI/CD
  View all use cases
- BY INDUSTRY
  Healthcare
  Financial services
  Manufacturing
  Government
  View all industries
View all solutions
- EXPLORE BY TOPIC
  AI
  Software Development
  DevOps
  Security
  View all topics
- EXPLORE BY TYPE
  Customer stories
  Events & webinars
  Ebooks & reports
  Business insights
  GitHub Skills
- SUPPORT & SERVICES
  Documentation
  Customer support
  Community forum
  Trust center
  Partners
View all resources
- COMMUNITY
  GitHub SponsorsFund open source developers
- PROGRAMS
  Security Lab
  Maintainer Community
  Accelerator
  GitHub Stars
  Archive Program
- REPOSITORIES
  Topics
  Trending
  Collections
- ENTERPRISE SOLUTIONS
  Enterprise platformAI-powered developer platform
- AVAILABLE ADD-ONS
  GitHub Advanced SecurityEnterprise-grade security features
  Copilot for BusinessEnterprise-grade AI features
  Premium SupportEnterprise-grade 24/7 support
Pricing

Search code, repositories, users, issues, pull requests...

Search

Clear

Search syntax tips

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Name

Query

To see all available qualifiers, see our documentation.

Appearance settings

You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.

Dismiss alert

glpi-project / glpi Public

Notifications You must be signed in to change notification settings
Fork 1.6k
Star 5.7k

Code
Issues 305
Pull requests 94
Actions
Projects
Security 155
Insights

Additional navigation options

Code
Issues
Pull requests
Actions
Projects
Security
Insights

Add validation for OS form#22913

Open

eduardomozart wants to merge 36 commits intoglpi-project:11.0/bugfixesglpi-project/glpi:11.0/bugfixesfrom

eduardomozart:add-validation-for-os-formeduardomozart/glpi:add-validation-for-os-formCopy head branch name to clipboard

Conversation Commits36 (36)Checks Files changed

Open

Add validation for OS form#22913
eduardomozart wants to merge 36 commits intoglpi-project:11.0/bugfixesfrom
eduardomozart:add-validation-for-os-form

Commits

Commits on Jan 30, 2026

Initial plan
Copilot
committed
Add validation to prevent empty OS records

Copilot
and
eduardomozart
committed
Fix prepareInputForUpdate to use current record data

Copilot
and
eduardomozart
committed
Add company, owner, and hostid to validation fields

Copilot
and
eduardomozart
committed
Address code review feedback: improve separation of concerns and simplify validation logic

Copilot
and
eduardomozart
committed
Merge pull request #7 from eduardomozart/copilot/add-validation-for-os-form
eduardomozart
authored
Update parameter type hint in areAllFieldsEmpty
eduardomozart
authored
Enhance OS addition tests with error message checks
eduardomozart
authored
Fix error message assertions in tests
eduardomozart
authored
Fix string quotes in error messages for consistency
eduardomozart
authored
Fix operating systems ID reference in tests
eduardomozart
authored
Fix string quotes in Item_OperatingSystemTest
eduardomozart
authored
Fix references for operating system and architecture
eduardomozart
authored
Update Item_OperatingSystem.php
eduardomozart
authored
Update item_operatingsystem.form.php
eduardomozart
authored
Change translation function from __() to __s()
eduardomozart
authored
Update Item_OperatingSystem.php
eduardomozart
authored
Replace prepareInputForUpdate to post_updateItem
eduardomozart
authored
Change test to verify record deletion on update
eduardomozart
authored
Clarify comment regarding update behavior in tests
eduardomozart
authored
Trim input fields before validation check
eduardomozart
authored
Rename post_updateItem to prepareInputForUpdate
eduardomozart
authored
Refactor OS test messages for consistency
eduardomozart
authored
Refactor OS add tests for better validation
eduardomozart
authored
Fix session message for empty operating system update
eduardomozart
authored
Improve error message clarity in OS tests
eduardomozart
authored
Refactor testUpdateOSToEmptyIsRejected method
eduardomozart
authored
Fix formatting in Item_OperatingSystemTest.php
eduardomozart
authored

Commits on Jan 31, 2026

Add test for capacity isUsed behavior
eduardomozart
committed
Use isUsed() and add capacity usage test
eduardomozart
committed
Adjust OS validation, messages, and tests
eduardomozart
committed
Prevent empty operating-system updates
eduardomozart
committed
Update Item_OperatingSystemTest.php
eduardomozart
committed
Update HasOperatingSystemCapacityTest.php
eduardomozart
committed
Fix numeric string validation in OS item
eduardomozart
committed
Fix logical condition in input validation
eduardomozart
authored

Footer

© 2026 GitHub, Inc.

Footer navigation

Terms
Privacy
Security
Status
Community
Docs
Contact

You can’t perform that action at this time.