Skip to content

Conversation

@bdalsass
Copy link
Contributor

Base information

Question Answer
Related to a SourceForge thead / Another PR / Combodo ticket? https://support.combodo.com/pages/UI.php?operation=details&class=Bug&id=8699
Type of change? Enhancement

Symptom (bug) / Objective (enhancement)

Attribute definition in PSR4 format

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have tested all changes I made on an iTop instance
  • I have added a unit test, otherwise I have explained why I couldn't
  • Is the PR clear and detailed enough so anyone can understand digging in the code?

@bdalsass bdalsass added this to the 3.3.0 milestone Sep 11, 2025
@bdalsass bdalsass self-assigned this Sep 11, 2025
@bdalsass bdalsass added core internal Work made by Combodo labels Sep 11, 2025
@bdalsass bdalsass force-pushed the feature/8699_attributedef_PSR4 branch 3 times, most recently from 5e7372f to f1d53e1 Compare September 12, 2025 09:28
- Duplicates attributedef.class.inc.php as attributedefrequires.class.inc.php (will contains the require_once directives)
- Duplicates attributedef.class.inc.php for each final class files AttributeApplicationLanguage.php, AttributeArchiveDate.php... to keep VCS
 history
- Remove attributedef.class.inc.php file
- Restore attributedef.class.inc.php with requires
- Update each attribute file code
@bdalsass bdalsass force-pushed the feature/8699_attributedef_PSR4 branch from f1d53e1 to a2c34fc Compare September 12, 2025 11:55
@Hipska
Copy link
Contributor

Hipska commented Sep 12, 2025

Won't this conflict with #741?

Also, all new files don't have an EOL on the latest line, see the ⛔ symbol in GitHub:
image

- Add namespaces
- Add use statements
- reformat code
@bdalsass bdalsass force-pushed the feature/8699_attributedef_PSR4 branch from a2c34fc to 041c30d Compare September 12, 2025 13:20
@bdalsass bdalsass merged commit 3fa500c into develop Sep 12, 2025
@Hipska
Copy link
Contributor

Hipska commented Sep 15, 2025

Too bad all these files are now added to the repo with missing EOL 😢

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core internal Work made by Combodo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants