Skip to content

feat[indexes]: Add support for inverted indexes #20

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.

Sign up for GitHub

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

Jump to bottom
Merged
lucassouzavieira merged 28 commits into from
Oct 11, 2025
Merged

feat[indexes]: Add support for inverted indexes #20

lucassouzavieira merged 28 commits into from
Oct 11, 2025

Conversation

@lucassouzavieira
Copy link
Owner

@lucassouzavieira lucassouzavieira commented Oct 11, 2025

Also includes some refactor work and code changes to fix php 8.4 deprecations

@lucassouzavieira
fix: use 'sprintf' instead of string interpolation
940ec00
@lucassouzavieira
drop tests for arango 3.11 and add tests with PHP 8.4
a4e2ae5
@lucassouzavieira
Update php.yml
608bf8e
@lucassouzavieira
Update dependencies
eb6ffb4
@lucassouzavieira
composer update + composer bump
50dd9b4
@lucassouzavieira
Small refactor on AQL namespace
8746ed9
@lucassouzavieira
update .gitignore
1ae675e
@lucassouzavieira
fix code sniffs
9ff23c0
@lucassouzavieira
Http namespace refactor
a7045fb
@lucassouzavieira
update dev dependencies
277529d
@lucassouzavieira
refactor: ODM tests
277745f
@lucassouzavieira
refactor: PSR12 code fix for tests
23b41d7
@lucassouzavieira
refactor: PSR12 code fix for src/
557a3ee
@lucassouzavieira
chore: remove obsolete collection properties (#14)
e82b6ce
@lucassouzavieira
Improved validation for collection key options (#15)
7cbddbc 
* improve Collection 'keyOptions' attribute validation

* refactor 'KeyType' class

* fix exception attribute type

* drop support for PHP 8.2
@lucassouzavieira
remove uses of 'print_r' (#16)
c4b9b19
@lucassouzavieira
Drop noop method 'load' for Collections (#17)
30671cd 
* small refactors for Cursor namespace

* drop no-op collection method 'load'

* refactor AQLFunction exception handling

* small refactors for AQL namespace
@lucassouzavieira
remove unused exception'
7a98143
@lucassouzavieira
remove unused variables
552c907
@lucassouzavieira
small refactor on Graph namespace
130d036
@lucassouzavieira
refactors on Transaction namespace
963d8f5
@lucassouzavieira
one more round of small fixes
31729ca
@lucassouzavieira
refactor array list
881660b
@lucassouzavieira
add make file with test, lint and static analyzer helpers
cb97abc
@lucassouzavieira
chore[PHP-8.4]: Handle php 8.4 deprecations (#18)
d168fdc 
* chore[php-8.4]: handle php 8.4 deprecations

* chore[php-8.4]: revert enforcing strict type on 'Admin\Task\Task::'
@lucassouzavieira
feat[index]: Add support for inverted indexes (#19)
5ab03ce 
* feat[inverted-index]: strict typing for base index class

* chore[indexes]: deprecate 'hash' and 'skiplist' index types

* feat[indexes]: base implementation for 'inverted' index

* tests[indexes]: remove ununsed imports

* tests[indexes]: add tests for inverted index

* feat[indexes]: add inverted index implementation
@lucassouzavieira
chore[documentation]: update existing docs
e9c36b5
@lucassouzavieira lucassouzavieira self-assigned this Oct 11, 2025
github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 11, 2025
View reviewed changes
docs/css/base.css

/* Colors */
--primary-color-hue: var(--color-hue-red, --color-hue-phpdocumentor-green);
--primary-color-hue: var(--color-hue-phpdocumentor-green, --color-hue-phpdocumentor-green);

Check warning

Code scanning / Csslint (reported by Codacy)

Expected RBRACE at line 44, col 5. Warning documentation

Expected RBRACE at line 44, col 5.
docs/css/base.css
ul.phpdocumentor-list {
margin-top: 0;
padding-left: var(--spacing-md);
padding-left: var(--spacing-lg);

Check notice

Code scanning / Csslint (reported by Codacy)

Expected () but found 'var(--spacing-lg)'. Note documentation

Expected () but found 'var(--spacing-lg)'.
docs/css/base.css
line-height: normal;
}

.phpdocumentor-on-this-page__content li.-deprecated {

Check notice

Code scanning / Csslint (reported by Codacy)

Element (li.-deprecated) is overqualified, just use .-deprecated without element name. Note documentation

Element (li.-deprecated) is overqualified, just use .-deprecated without element name.
github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 11, 2025
View reviewed changes
docs/js/template.js
})();
function openSvg(svg) {
// convert to a valid XML source
const as_text = new XMLSerializer().serializeToString(svg);

Check notice

Code scanning / Jshint (reported by Codacy)

Prohibits the use of __iterator__ property due to compatibility issues Note documentation

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
docs/js/template.js
// convert to a valid XML source
const as_text = new XMLSerializer().serializeToString(svg);
// store in a Blob
const blob = new Blob([as_text], { type: "image/svg+xml" });

Check notice

Code scanning / Jshint (reported by Codacy)

Prohibits the use of __iterator__ property due to compatibility issues Note documentation

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
docs/js/template.js
// store in a Blob
const blob = new Blob([as_text], { type: "image/svg+xml" });
// create an URI pointing to that blob
const url = URL.createObjectURL(blob);

Check notice

Code scanning / Jshint (reported by Codacy)

Prohibits the use of __iterator__ property due to compatibility issues Note documentation

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
docs/js/template.js
const blob = new Blob([as_text], { type: "image/svg+xml" });
// create an URI pointing to that blob
const url = URL.createObjectURL(blob);
const win = open(url);

Check notice

Code scanning / Jshint (reported by Codacy)

Prohibits the use of __iterator__ property due to compatibility issues Note documentation

'const' is available in ES6 (use 'esversion: 6') or Mozilla JS extensions (use moz).
docs/js/template.js

var svgs = document.querySelectorAll(".phpdocumentor-uml-diagram svg");
for( var i=0,il = svgs.length; i< il; i ++ ) {
svgs[i].onclick = (evt) => openSvg(evt.target);

Check notice

Code scanning / Jshint (reported by Codacy)

Prohibits the definition of function inside the body of loops Note documentation

Functions declared within loops referencing an outer scoped variable may lead to confusing semantics. (openSvg)
github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 11, 2025
View reviewed changes
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Phpcs (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 11, 2025
View reviewed changes
docs/css/base.css

/* Colors */
--primary-color-hue: var(--color-hue-red, --color-hue-phpdocumentor-green);
--primary-color-hue: var(--color-hue-phpdocumentor-green, --color-hue-phpdocumentor-green);

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning documentation

Expected indentation of 2 spaces (indentation)
docs/css/base.css
/* Base Styles
-------------------------------------------------- */
body {
background-color: #fff;

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning documentation

Expected indentation of 2 spaces (indentation)
docs/css/base.css
word-break: break-all;
line-height: normal;
}

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning documentation

Expected indentation of 2 spaces (indentation)
docs/css/base.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning documentation

Expected indentation of 2 spaces (indentation)
docs/css/base.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 4 spaces (indentation) Warning documentation

Expected indentation of 4 spaces (indentation)
docs/css/template.css
.phpdocumentor-table-of-contents .phpdocumentor-table-of-contents__entry.-constant:before {
content: 'C';
background-color: transparent;
background-image: url('data:image/svg+xml;utf8,<svg width="20" height="20" viewBox="0 0 20 20" fill="none" xmlns="http://www.w3.org/2000/svg"><rect x="-3.05176e-05" y="9.99998" width="14.1422" height="14.1422" transform="rotate(-45 -3.05176e-05 9.99998)" fill="hsl( 4, 57%, 60%)"/></svg>');

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning documentation

Expected indentation of 2 spaces (indentation)
docs/css/template.css
.phpdocumentor-tag-link {
margin-right: var(--spacing-sm);
}
.phpdocumentor-uml-diagram svg {

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning documentation

Expected indentation of 2 spaces (indentation)
docs/css/template.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected empty line before rule (rule-empty-line-before) Warning documentation

Expected empty line before rule (rule-empty-line-before)
docs/css/template.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Expected indentation of 2 spaces (indentation) Warning documentation

Expected indentation of 2 spaces (indentation)
docs/css/template.css Outdated

Check warning

Code scanning / Stylelint (reported by Codacy)

Unexpected missing end-of-source newline (no-missing-end-of-source-newline) Warning documentation

Unexpected missing end-of-source newline (no-missing-end-of-source-newline)
@lucassouzavieira lucassouzavieira merged commit 30b5560 into main Oct 11, 2025
12 checks passed
@lucassouzavieira lucassouzavieira deleted the develop branch October 11, 2025 20:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@lucassouzavieira lucassouzavieira

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lucassouzavieira